[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