Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Integrated arm assembler doesn't understand vand.i32 q10,#0x03ffffff #21076

Closed
nico opened this issue Aug 18, 2014 · 6 comments
Closed

Integrated arm assembler doesn't understand vand.i32 q10,#0x03ffffff #21076

nico opened this issue Aug 18, 2014 · 6 comments
Assignees
@rengolin
Labels
backend:ARM bugzilla Issues migrated from bugzilla

Comments

@nico
Copy link
Contributor

nico commented Aug 18, 2014

Bugzilla Link 20702
Resolution FIXED
Resolved on Sep 25, 2014 06:54
Version trunk
OS All
Blocks #20796
CC @compnerd,@rengolin

Extended Description

../../third_party/boringssl/src/crypto/poly1305/poly1305_arm_asm.S:1167:14: error: invalid operand for instruction
vand.i32 q10,#0x03ffffff
^
../../third_party/boringssl/src/crypto/poly1305/poly1305_arm_asm.S:1191:13: error: invalid operand for instruction
vand.i32 q9,#0x03ffffff
^

compnerd says this is valid, so IAS should support it.
@nico
Copy link
Contributor Author

nico commented Aug 18, 2014

assigned to @rengolin

@rengolin
Copy link
Member

VAND immediate is a pseudo-instruction equivalent to VBIC with the inverted immediate. We should be able to add a pseudo to the table-gen.

@nico
Copy link
Contributor Author

nico commented Sep 7, 2014

Any progress?

@rengolin
Copy link
Member

rengolin commented Sep 7, 2014

Hi Nico,

Haven't got to this one yet. I'm assigning to myself so that I see on my list, hoping to have a look at it next week.

cheers,
--renato

@rengolin
Copy link
Member

rengolin commented Sep 9, 2014

http://reviews.llvm.org/D5263

@rengolin
Copy link
Member

Fixed in r218450.

@nico nico mentioned this issue Jul 24, 2014
Closed
@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 9, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rengolin rengolin

Labels
backend:ARM bugzilla Issues migrated from bugzilla
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nico @rengolin