[PATCH] D142217: [MC] Store target Insts table in reverse order. NFC.

Sergei Barannikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 23 13:13:27 PST 2023


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

LGTM



================
Comment at: llvm/utils/TableGen/InstrInfoEmitter.cpp:935
+  unsigned Num = NumberedInstructions.size();
+  for (const CodeGenInstruction *Inst : llvm::reverse(NumberedInstructions)) {
     // Keep a list of the instruction names.
----------------
nit: `reverse` should be visible without `llvm::` qualification.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142217



More information about the llvm-commits mailing list