[PATCH] D96914: [ELF] Add -z start-stop-gc to let __start_/__stop_ not retain C identifier name sections

Fri Nov 12 11:06:43 PST 2021

jrtc27 added a comment.

In D96914#3127982 <https://reviews.llvm.org/D96914#3127982>, @tstellar wrote:

> @MaskRay To be clear, I'm talking about reverting 6d2d3bd0a61f5fc7fd9f61f48bc30e9ca77cc619 <https://reviews.llvm.org/rG6d2d3bd0a61f5fc7fd9f61f48bc30e9ca77cc619> not this patch.  Again, I understand your motivations for keeping the default as-is, but we don't have consensus on this change and the policy for this project is to revert until a consensus can be reached.  Also, the fact the this patch was approved on the condition that the default would stay the same, but then the default was changed without discussion is a pretty strong reason to revert.  Can we please revert 6d2d3bd0a61f5fc7fd9f61f48bc30e9ca77cc619 <https://reviews.llvm.org/rG6d2d3bd0a61f5fc7fd9f61f48bc30e9ca77cc619> in trunk and continue the discussions in this thread?
>
> @jrtc27 @hvdijk  Do you have a suggested timeline for transitioning to the new default?

My instinct would be something like a couple of years from the date the new flags first appeared in an LLVM release, given that's the rough release cycle length for many Linux distros., so targeting LLVM 17 assuming the current cadence continues; anything less than that and your software won't be able to support LLD on even the latest release of some distros.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D96914/new/

https://reviews.llvm.org/D96914