<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">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>-eric</div></div></div>