[PATCH] D90009: [X86] VEX/EVEX prefix doesn't work for inline assembly.
Craig Topper via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 22 22:43:12 PDT 2020
craig.topper added inline comments.
================
Comment at: llvm/lib/Target/X86/MCTargetDesc/X86InstPrinterCommon.cpp:354
+ else if (Flags & X86::Force_VEX2Encoding)
+ O << "\t{vex2}";
+ else if (Flags & X86::Force_VEX3Encoding)
----------------
craig.topper wrote:
> Is it important that we use {vex2} instead of just treating it as {vex}?
I guess we'd actually have to print {vex2} instead of {vex} for compatibility with older versions of GNU assember that don't support {vex}. So I guess maybe they should be different as much as I hate wasting flags
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D90009/new/
https://reviews.llvm.org/D90009
More information about the llvm-commits
mailing list