<div dir="ltr">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))<br><br>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.<br><br>Though I also think probably the right answer here is not to pick one of the two locations, but to drop the location entirely.<br><br>- Dave</div><br><div class="gmail_quote"><div dir="ltr">On Fri, Aug 24, 2018 at 4:31 PM Phabricator via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This revision was not accepted when it landed; it landed in state "Needs Review".<br>
This revision was automatically updated to reflect the committed changes.<br>
Closed by commit rL340672: Prevent DILocation::getMergedLocation() from creating invalid metadata. (authored by adrian, committed by ).<br>
<br>
Changed prior to commit:<br>
  <a href="https://reviews.llvm.org/D51238?vs=162492&id=162499#toc" rel="noreferrer" target="_blank">https://reviews.llvm.org/D51238?vs=162492&id=162499#toc</a><br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D51238" rel="noreferrer" target="_blank">https://reviews.llvm.org/D51238</a><br>
<br>
Files:<br>
  llvm/trunk/lib/IR/DebugInfoMetadata.cpp<br>
  llvm/trunk/unittests/IR/MetadataTest.cpp<br>
<br>
</blockquote></div>