<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Aug 4, 2020 at 11:38 PM Eric Christopher <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Aug 4, 2020 at 11:32 PM Fangrui Song 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:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">MaskRay added a comment.<br>
<br>
> At which point we are at an impasse and the default behavior applies. Please revert completely back to the original behavior immediately.<br>
><br>
> Thanks.<br>
<br>
With respect I think the "request for changes" blocked the change and I am not inclined to agree we would otherwise be at an impasse without the "request for changes".<br>
I would likely get an "Accept Revision" or a just textual "LGTM", even if there is a conditional request that ".debug_* should use 0 as well".<br></blockquote><div> </div><div>Right now it appears that we can agree on a forward direction to make or even which behavior to revert to so that means that we need to revert all the way.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
I sent the patch instead of reverting 3 or 4 dependent patches because I think reverting will just make the situation more confusing. I have experienced the review process and have read all the context,<br>
so I very responsibly state again that "a simple revert of 3 or 4 dependent patches will just make the situation more confusing".<br></blockquote><div><br></div><div>Sadly it might in the short term make things more confusing, but it gives our users a place to start from. </div><div><br></div><div>That this thread has gone on so long and currently our users are blocked on being able to do anything means we need to go back to a default state whether or not that's less convenient for us with changes in behavior.</div><div><br></div></div></div></blockquote><div><br></div><div>To elaborate further: I'm sympathetic to wanting to get the behavior a little more set, but unfortunately it looks like we haven't been able to come to agreement on what that behavior is.  My personal thought is to go with switching to the bfd semantics and maybe fix debug_loc the same way as debug_ranges to be a bit better than the default bfd semantics.<br><br>I think if that direction is ok with you for now then we can go ahead and make that change otherwise I think we need to reset and try to make a plan to move forward again.</div><div><br></div><div>Thanks and sorry it's so much discussion here :)</div><div><br></div><div>-eric</div></div></div>