[PATCH] D27471: [BDCE/DebugInfo] Preserve llvm.dbg.value's argument

[Davide Italiano via Phabricator via llvm-commits]

davide added inline comments.


================
Comment at: test/Transforms/BDCE/pr26587.ll:10
+; CHECK-NEXT: tail call void (...) @h()
+; CHECK-NEXT: %call = tail call i32 (...) @g(), !dbg !10
+; CHECK-NEXT: tail call void @llvm.dbg.value(metadata i32 %call
----------------
aprantl wrote:
> Does this also work with a non-asserts opt? I'm not sure if it is clang that is stripping out the values' names or if the parser does that, too.
> 
> `CHECK-NEXT: %[[CALL:.*]] = tail call i32 (...) @g()`
> 
> Probably don't need to check for the !dbg !10 here.
Done. Confirmed that the test passes also with Release (non assert)


https://reviews.llvm.org/D27471