[PATCH] D121402: [X86][NFC] Move table from getRelaxedOpcodeArith into its own class

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 11 22:05:39 PST 2022


MaskRay accepted this revision.
MaskRay added inline comments.


================
Comment at: llvm/lib/Target/X86/MCTargetDesc/X86InstrRelaxTables.cpp:132
+
+    sort(Table.begin(), Table.end());
+
----------------
`llvm::sort(Table)`


================
Comment at: llvm/lib/Target/X86/MCTargetDesc/X86InstrRelaxTables.cpp:135
+    // Now that it's sorted, ensure its unique.
+    assert(is_sorted(Table) && "Short form table is not unique!");
+  }
----------------
llvm::is_sorted if an LLVM STLExtras util may conflict with std:: of the same name


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121402



More information about the llvm-commits mailing list