[PATCH] D48312: [X86] Use binary search of the EVEX->VEX static tables instead of populating two DenseMaps for lookups

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 19 14:40:55 PDT 2018


RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.

LGTM with one minor



================
Comment at: lib/Target/X86/X86EvexToVex.cpp:263
 
+  unsigned NewOpc = I->VexOpcode;
+
----------------
Move this to just above the performCustomAdjustments call?


https://reviews.llvm.org/D48312





More information about the llvm-commits mailing list