<div dir="ltr">Ping - any luck?</div><br><div class="gmail_quote"><div dir="ltr">On Mon, Aug 27, 2018 at 12:42 PM Adrian Prantl <<a href="mailto:aprantl@apple.com">aprantl@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word;line-break:after-white-space"><div><br><blockquote type="cite"><div>On Aug 27, 2018, at 11:54 AM, David Blaikie <<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>> wrote:</div><br class="m_4103782738803242710Apple-interchange-newline"><div><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></div></div></blockquote><div><br></div></div></div><div style="word-wrap:break-word;line-break:after-white-space"><div><div>One of the testcases I had was the *entire* Swift standard library and unfortunately bugpoint was not helpful at reducing the IR. In Swift code it's possible to end up with thunks with no debug information, so that could fit the description. I'll see if I can find a smaller example that I can actually share.<br></div></div></div><div style="word-wrap:break-word;line-break:after-white-space"><div><div><br></div><div>-- adrian</div></div></div><div style="word-wrap:break-word;line-break:after-white-space"><div><div><br></div><blockquote type="cite"><div><div dir="ltr"><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" target="_blank">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>
</div></blockquote></div></div></blockquote></div>