<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 9, 2015 at 2:26 PM, Adrian Prantl <span dir="ltr"><<a href="mailto:aprantl@apple.com" target="_blank">aprantl@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">> Try it - it shouldn't assert. The assertion only fires when the caller has debug info as well.<br>
<br>
</span>Interesting! I tried a triple-nested call were the middle function was without debug info and the inlined instructions from the function without debug info get re-parented into the callee function and it works out. Sweet.<br>
<br>
I’m retracting my concerns. I will look into fixing CFE to emit valid DebugLocs inside the landingpad.<br></blockquote><div><br>Nothing to do on your end - the absence of a DebugLoc was introduced by this patch, which I'm reverting. Then I'll figure out how to commit it without introducing that (& several other, judging by the reproductions provided), bugs.<br><br>Though I'm certainly happy for ideas/pointers, don't feel that there's any work for you to do to unblock/fix this - I broke it.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
<br>
> 1) a function with debug info calls<br>
> 2) a function with instructions with debuglocs (not necessarily a function with debug info - it could be a nodebug function that's had debug functions inlined into it already) through<br>
> 3) a call without a debug location<br>
<br>
</span>Could you craft this into an assertion and add it — maybe to the inliner? Alternatively, could you add this as a comment next to the assertion?<br></blockquote><div><br>I wouldn't mind doing so, though I haven't given it a great deal of thought - as you can see, the cases that are accepted (intervening nodebug functions, etc) wrinkle this a bit.<br><br>But it should boil down to "find this function in the debug info subprogram list, if found, scan the inlined instructions for debuglocs, if found, assert that the call instruction has a debug location".<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
thanks,<br>
adrian</blockquote></div><br></div></div>