[PATCH] D62669: [ARM] Set up infrastructure for MVE vector instructions.
Dave Green via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 4 12:46:01 PDT 2019
dmgreen added inline comments.
================
Comment at: llvm/lib/Target/ARM/AsmParser/ARMAsmParser.cpp:2183
+ unsigned NextOpIndex = Inst.getNumOperands();
+ const MCInstrDesc &MCID = ARMInsts[Inst.getOpcode()];
+ int TiedOp = MCID.getOperandConstraint(NextOpIndex, MCOI::TIED_TO);
----------------
I think this should be using MII.get(Inst.getOpcode()), if it can.
================
Comment at: llvm/lib/Target/ARM/MCTargetDesc/ARMMCCodeEmitter.cpp:535
+ if (STI.getFeatureBits()[ARM::HasMVEIntegerOps] ||
+ STI.getFeatureBits()[ARM::HasMVEFloatOps])
+ return RegNo;
----------------
Does HasMVEFloatOps imply HasMVEIntegerOps?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D62669/new/
https://reviews.llvm.org/D62669
More information about the llvm-commits
mailing list