[PATCH] D55682: [ELF] Add -z start-stop-visibility= to set __start_/__stop_ symbol visibility

Petr Hosek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 17 23:58:13 PDT 2020


phosek added a comment.

In D55682#2099923 <https://reviews.llvm.org/D55682#2099923>, @MaskRay wrote:

> LGTM. @pcc and @ruiu objected to this patch. Please give some time for them to make comments. Friday is a no meeting day for some of us, so waiting til next week will be nice.


SGTM thanks! I'm also fine waiting until the GNU change lands.

> (I guess @grimar may want you to use different output files for different RUN lines.)

Using different output files is fine with me.


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

https://reviews.llvm.org/D55682





More information about the llvm-commits mailing list