[PATCH] D27901: [X86][[AVX512] Code size reduction in X86 by replacing EVEX with VEX encoding

Gadi Haber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 27 07:48:08 PST 2016


gadi.haber marked an inline comment as done.
gadi.haber added inline comments.


================
Comment at: lib/Target/X86/X86EvexToVex.cpp:219
+    MI.setDesc(MCID);
+    MI.setAsmPrinterFlag((MachineInstr::CommentFlag)AC_EXEX_2_VEX);
+    Changed = true;
----------------
gadi.haber wrote:
> zvi wrote:
> > Now that this API takes also target-specific flags, I think that instead of casting, setAsmPrinterFlag(CommentFlag) (and friends) should be changed to setAsmPrinterFlag(uint8_t) or something similar.
> There is only single definition of setAsmPrinterFlag in MachineInstr.h and it receives the CommenFlag parameter.
> 
OK. I understand your comment now.
I changed the definition of SetAsmPrinterFlag(CommentFlag) to SetAsmPrinterFlag(unit8_t)  in MachineInstr.h



Repository:
  rL LLVM

https://reviews.llvm.org/D27901





More information about the llvm-commits mailing list