[PATCH] D46338: [GlobalISel][Legalizer] LegalizerInfo verifier: checking that legalization rules cover all type indices

Amara Emerson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 28 10:37:28 PDT 2018


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

Overall I'm in favour of this change, so LGTM.



================
Comment at: lib/CodeGen/GlobalISel/LegalizerInfo.cpp:553
+        });
+    DEBUG(dbgs() << MII.getName(Opcode) << " (opcode " << Opcode
+                 << "): " << NumTypeIdxs << " type ind"
----------------
This will need to be LLVM_DEBUG now I think.


Repository:
  rL LLVM

https://reviews.llvm.org/D46338





More information about the llvm-commits mailing list