[PATCH] D23040: Tie the Verifier class to a Module; NFCI

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 1 17:16:06 PDT 2016


mehdi_amini added a subscriber: mehdi_amini.

================
Comment at: lib/IR/Verifier.cpp:310
@@ -314,1 +309,3 @@
+    assert(&M == this->M &&
+           "An instance of this class only works with a specific module!");
     Context = &M.getContext();
----------------
I'd just remove the Module parameter as part of the same patch. It becomes a sketchy invariant to be part of the API now.


https://reviews.llvm.org/D23040





More information about the llvm-commits mailing list