[PATCH] D27492: [DWARF] Suppress .loc directives coming from CFI instructions

[Paul Robinson via Phabricator via llvm-commits]

probinson added inline comments.


================
Comment at: test/DebugInfo/Generic/no-cfi-loc.ll:2
+; RUN: %llc_dwarf -filetype=asm -use-unknown-locations < %s | FileCheck %s
+; CHECK-NOT: .loc 1 0 0
+
----------------
aprantl wrote:
> Short comment on what is being checked here?
> And is there anything positive that could be checked for, too? Tests that only CHECK-NOT love to break without anybody noticing :-)
It ends up emitting a CFI instruction with no debug location. The test is verifying that this does not result in a line-0 record (that we skip the CFI instruction).

Although, I now realize the fact that there is a CFI instruction at all is not obvious!  Maybe I should turn this into an MIR test instead.


https://reviews.llvm.org/D27492