[PATCH] D78177: [VE] Update logical operation instructions
Kazushi Marukawa via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 15 01:36:07 PDT 2020
kaz7 marked an inline comment as done.
kaz7 added inline comments.
================
Comment at: llvm/lib/Target/VE/VEInstrInfo.td:913
+let cw = 1, cw2 = 1 in defm CMOVS : RRCMOVm<"cmov.s.${cfw}", 0x3B, F32, f32>;
+def : MnemonicAlias<"cmov.l", "cmov.l.at">;
+def : MnemonicAlias<"cmov.w", "cmov.w.at">;
----------------
simoll wrote:
> First time i am seeing this feature. Do we need this to match mnemonic aliases when parsing assembly?
Yes. With this MenmonicAlias, llvm-mc can parse "cmov.l" as a shorten "cmov.l.at" instruction. NEC assembler allows both of them, so we also allow them to maximize user's choices.
May I defer these to coming llvm-mc patch sets?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78177/new/
https://reviews.llvm.org/D78177
More information about the llvm-commits
mailing list