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

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 25 11:34:02 PST 2018


aprantl added a comment.

Thanks for checking!

Based on this, we could either

- make it the responsibility of the Caller of LLParser to run the Verifier and update those tools to run the Verifier.
- or pass the override datalayout into LLParser so it can inject it before running the Verifier.

I personally don't like the idea of relaxing the Verifier / making it verify something that is not what is being fed into the later stages. (Feel free to try and convince me otherwise though :-)


https://reviews.llvm.org/D41832





More information about the llvm-commits mailing list