[PATCH] D90009: [X86] VEX/EVEX prefix doesn't work for inline assembly.

LiuChen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 23 01:13:06 PDT 2020


LiuChen3 added inline comments.


================
Comment at: llvm/lib/Target/X86/MCTargetDesc/X86InstPrinterCommon.cpp:352
+  if (Flags & X86::Force_VEXEncoding)
+    O << "\t{vex}";
+  else if (Flags & X86::Force_VEX2Encoding)
----------------
pengfei wrote:
> `"\t{vex}\t"` ?
No "\t" needed to add to the end of prefix. The printer will handle it correctly. 


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