[PATCH] D111538: [X86] Prefer VEX encoding in X86 assembler.

[LuoYuanke via Phabricator via llvm-commits]

LuoYuanke added inline comments.


================
Comment at: llvm/utils/TableGen/AsmMatcherEmitter.cpp:639
 
-    return false;
+    return TheDef->getID() < RHS.TheDef->getID();
   }
----------------
skan wrote:
> LuoYuanke wrote:
> > Is there any approach to check it is X86 target?
> Sure. You can check `TheDef->isSubClassOf("X86Inst")`
Great! I'll update the patch.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D111538/new/

https://reviews.llvm.org/D111538