[PATCH] D84825: [release/11.x only][ELF] Change tombstone value -1 to 0

[Eric Christopher via llvm-commits]

On Tue, Aug 4, 2020 at 11:32 PM Fangrui Song via Phabricator <
reviews at reviews.llvm.org> wrote:

> MaskRay added a comment.
>
> > At which point we are at an impasse and the default behavior applies.
> Please revert completely back to the original behavior immediately.
> >
> > Thanks.
>
> 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".
> 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".
>

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.

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,
> so I very responsibly state again that "a simple revert of 3 or 4
> dependent patches will just make the situation more confusing".
>

Sadly it might in the short term make things more confusing, but it gives
our users a place to start from.

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.

-eric
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200804/62055dbb/attachment.html>