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

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 20 00:08:31 PST 2016


craig.topper added inline comments.


================
Comment at: lib/Target/X86/X86EvexToVex.cpp:47
+
+    /// For EVEX instructions that can be encoded using VEX encodig, replace 
+    /// them by the VEX encoding in order to reduce size.
----------------
This also says "encodig"


================
Comment at: lib/Target/X86/X86EvexToVex.cpp:130
+
+// For EVEX instructions that can be encoded using VEX encodig, 
+// replace them by the VEX encoding in order to reduce size:
----------------
gadi.haber wrote:
> zvi wrote:
> > *encoding
> Did you mean removing the comma that follows encoding?
The first line of the comment says "encodig" rather than "encoding"


Repository:
  rL LLVM

https://reviews.llvm.org/D27901





More information about the llvm-commits mailing list