[PATCH] D41832: LLParser: do not verify LLVM module

Yaxun Liu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 25 11:08:13 PST 2018


yaxunl added a comment.

In https://reviews.llvm.org/D41832#988226, @aprantl wrote:

> > LLParser already calls Verifier before that by itself
>
> I wonder if that is actually necessary. Are there any code paths that invoke LLParser and don't run the Verifier afterwards? If not, then it would make sense to change that.


There are some llvm tools invoke LLParser but do not run the Verifier afterwards, e.g. lli, llvm-cat, llvm-diff, etc.


https://reviews.llvm.org/D41832





More information about the llvm-commits mailing list