[PATCH] Add aliases for VAND imm to VBIC ~imm

Renato Golin renato.golin at linaro.org
Tue Sep 9 07:32:48 PDT 2014


Right, ignoring the code in isNEONi32splat() for now, I just want to make sure everything is in the right place.

I have:
- removed the unnecessary print methods
- moved validateNEON... to ARMAddressingModes.h as encodeNEON...
- added a common isNEON to the same header and inverted the value on ...SplatNot methods

I haven't:
- fixed isNEON...
- added enough tests to make sure a whole range of values are encoded correctly

I don't know if:
- isNEON... is a good name, since that it's now just checking for splat consistency and changing it. Maybe encodeNEON...Value() would be better?

http://reviews.llvm.org/D5263

Files:
  lib/Target/ARM/ARMInstrNEON.td
  lib/Target/ARM/AsmParser/ARMAsmParser.cpp
  lib/Target/ARM/MCTargetDesc/ARMAddressingModes.h
  test/MC/ARM/neon-bitwise-encoding.s
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D5263.13462.patch
Type: text/x-patch
Size: 8220 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140909/04da7d98/attachment.bin>


More information about the llvm-commits mailing list