[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