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

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: test/Verifier/llvm.dbg.intrinsic-dbg-attachment.ll:1
-; RUN: not llvm-as -disable-output <%s 2>&1 | FileCheck %s
+; RUN: llvm-as -disable-output <%s 2>&1 | FileCheck %s
 define void @foo() {
----------------
yaxunl wrote:
> arsenm wrote:
> > This is a test regression
> This test should pass since it only contains invalid debug info. After the parser removes the debug info, the IR is valid. There is a bug in verifier mistakenly treated debug info error as non-debug info error. (See change in Verifier.cpp line 4492 for the fix). After that bug is fixed, this test should pass.
Can you fix this in a separate patch then?


https://reviews.llvm.org/D41832