[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