Re: [PATCH v2] Add -fgnu-retain to place used symbols in SHF_GNU_RETAIN section

2021-02-18 Thread Richard Biener via Gcc-patches
On Wed, Feb 17, 2021 at 3:33 PM Jakub Jelinek via Gcc-patches wrote: > > On Tue, Feb 16, 2021 at 11:59:21AM -0800, H.J. Lu wrote: > > PR target/99113 > > * common.opt: Add -fgnu-retain. > > I'm not sure -fgnu-retain as the option name. > Wouldn't say -fretain-used-vars be better? -fre

Re: [PATCH v2] Add -fgnu-retain to place used symbols in SHF_GNU_RETAIN section

2021-02-17 Thread Jakub Jelinek via Gcc-patches
On Tue, Feb 16, 2021 at 11:59:21AM -0800, H.J. Lu wrote: > PR target/99113 > * common.opt: Add -fgnu-retain. I'm not sure -fgnu-retain as the option name. Wouldn't say -fretain-used-vars be better? > @@ -1666,6 +1666,10 @@ floop-unroll-and-jam > Common Var(flag_unroll_jam) Optimizati

[PATCH v2] Add -fgnu-retain to place used symbols in SHF_GNU_RETAIN section

2021-02-16 Thread H.J. Lu via Gcc-patches
f the configured assembler/linker doesn't support those > 3) add a non-default option through which one could opt in for "used" >attribute to imply retain attribute too for projects that want that? > > Jakub > Here is the updated patch to #1 and #3. I will