[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.

Superficially (I haven't looked at all the codepaths, etc) the code change looks OK.

Testing could probably be a lot simpler - can you take the merged IR and test that, rather than running LTO steps in the test? It seems the resulting IR is what we want to test/what's being fixed, not the path that was used to reach it.


https://reviews.llvm.org/D29765