[llvm] [X86][NF] Switch the order of Inst and &Target.getInstruction(NewRec) (PR #130739)

Shengchen Kan via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 11 07:55:07 PDT 2025


================
@@ -3304,10 +3304,28 @@ static unsigned getNewOpcFromTable(ArrayRef<X86TableEntry> Table,
   return (I == Table.end() || I->OldOpc != Opc) ? 0U : I->NewOpc;
 }
 unsigned X86::getNFVariant(unsigned Opc) {
+#ifndef NDEBUG
----------------
KanRobert wrote:

Can we only enable it in expensive check? It's a big overhead to check the whole table for each lookup

https://github.com/llvm/llvm-project/pull/130739


More information about the llvm-commits mailing list