[PATCH] D51238: Add a unit test for DILocation::getMergedLocation()

[David Blaikie via Phabricator via llvm-commits]

dblaikie added a subscriber: aprantl.
dblaikie added a comment.

Could you still come up with a practical test case for this? (doesn't
necessarily need to be checked in, to be honest - but something to validate
that this use case/support is needed & not working around a bug (& if it is
working around a bug, that we know about it so we can remove the workaround
eventually & not leave it in as folklore and cruft))

I can imagine that there could be cases where there would be no common
scope - due to inlining into a nodebug function (we have that classic
assertion to ensure all scope chains lead to the DISubprogram that contains
those locations - but that does not apply if there is no DISubprogram for a
function - if it is nodebug and has had debug info inlined into it (which
is still useful if, then, the nodebug function is inlined into somewhere
else)). So maybe you can search around there.

Though I also think probably the right answer here is not to pick one of
the two locations, but to drop the location entirely.

- Dave


Repository:
  rL LLVM

https://reviews.llvm.org/D51238