[PATCH] D29765: Handle link of NoDebug CU with a CU that has debug emission enabled

[David Blaikie via Phabricator via llvm-commits]

dblaikie added a comment.

Are the locations properly respected/ignored in this case? (what does the line table end up looking like for the instructions in the nodebug inlined code?)

Would it be worth/possible omitting these scopes from the LexicalScope tree in the first place, rather than filtering out here after-the-fact?



================
Comment at: test/DebugInfo/Generic/debug_and_nodebug_CUs.ll:3
+; debug is handled correctly.
+
+; RUN: llc %s -o - | FileCheck %s
----------------
Include comments about the source code and commands to reproduce the IR (since debug info metadata is a bit hard to read, this can help with understanding exactly what the test is testing, reproducing, etc)


https://reviews.llvm.org/D29765