[PATCH] [ARMv8] NEON instruction VRINT{N, X, A, Z, M, P}

Joey Gouly joey.gouly at arm.com
Thu Jul 18 06:13:52 PDT 2013


Hi t.p.northover,

Fairly standard patch.

As the comment says, VRINT{Z, R, X} have a predicate operand in VFP but not in NEON, so I had to special case it in 'shouldOmitCCOutOperand'. However, it's a predicate that I'm not deleting, not a CCOut. Should I create a new function (shouldOmitPredicateOperand), rename the current one, or leave it as is?

http://llvm-reviews.chandlerc.com/D1175

Files:
  test/MC/ARM/neon-v8.s
  test/MC/ARM/thumb-neon-v8.s
  test/MC/Disassembler/ARM/neon-v8.txt
  test/MC/Disassembler/ARM/thumb-neon-v8.txt
  test/CodeGen/ARM/vrint.ll
  include/llvm/IR/IntrinsicsARM.td
  lib/Target/ARM/ARMInstrNEON.td
  lib/Target/ARM/AsmParser/ARMAsmParser.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D1175.1.patch
Type: text/x-patch
Size: 12601 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130718/dcb14c36/attachment.bin>


More information about the llvm-commits mailing list