On 01/02/2016 08:16 PM, Marcin Kościelnicki wrote: > Previously, .rodata was hardcoded. For C++ vague linkage functions, > this resulted in needlessly duplicated literals. With the new split > stack support, this resulted in link errors, due to .rodata containing > relocations to the discarded text sections. > > gcc/ChangeLog: > > * config/s390/s390.md (pool_section_start): Use switch_to_section > to select proper read-only data section instead of hardcoding .rodata. > (pool_section_end): Use switch_to_section to match the above. > --- > gcc/ChangeLog | 6 ++++++ > gcc/config/s390/s390.md | 11 +++++++++-- > 2 files changed, 15 insertions(+), 2 deletions(-) > > diff --git a/gcc/ChangeLog b/gcc/ChangeLog > index 23ce209..2c572a7 100644 > --- a/gcc/ChangeLog > +++ b/gcc/ChangeLog > @@ -1,3 +1,9 @@ > +2016-01-02 Marcin Kościelnicki <koria...@0x04.net> > + > + * config/s390/s390.md (pool_section_start): Use switch_to_section > + to select proper read-only data section instead of hardcoding .rodata. > + (pool_section_end): Use switch_to_section to match the above. > +
This is ok if bootstrap and regression tests are clean. Thanks! -Andreas-
