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

[Yaxun Liu via Phabricator via llvm-commits]

yaxunl added a comment.

I tried to add a parameter to LLParser for overriding datalayout. However I encountered another issue for llc.

If target is not specified on llc command line, llc needs to parse the LLVM assembly to get the target. To parse the LLVM assembly, llc needs to pass the datalayout to LLParser first. However, it does not know the datalayout since it does not know the target.

It seems we have to go back the current approach, that is, let LLParser only checks debug info and ignores non-debug info. For those llvm tools which does not explicitly verify module after parsing, I can add explicit verification after parsing.


https://reviews.llvm.org/D41832