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

[Yaxun Liu via Phabricator via llvm-commits]

yaxunl 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:
> > 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?
> will do.
Separated the fix to https://reviews.llvm.org/D42391


https://reviews.llvm.org/D41832