procyberian/gcc
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

S/390: Use proper read-only data section for literals.

Browse source 
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.

From-SVN: r232667
This commit is contained in:
Marcin Kościelnicki 2016-01-21 09:45:42 +00:00 • committed by Andreas Krebbel
parent fac40b023f
commit b929b47014
2 changed files with 16 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
gcc/ChangeLog
View file

@ -1,3 +1,10 @@
2016-01-21 Marcin Kościelnicki <koriakin@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.
2016-01-21 Richard Biener <rguenther@suse.de>
PR tree-optimization/69378

11
gcc/config/s390/s390.md
View file

@ -10247,13 +10247,20 @@
(define_insn "pool_section_start"
[(unspec_volatile [(const_int 1)] UNSPECV_POOL_SECTION)]
""
".section\t.rodata"
{
switch_to_section (targetm.asm_out.function_rodata_section
(current_function_decl));
return "";
}
[(set_attr "length" "0")])
(define_insn "pool_section_end"
[(unspec_volatile [(const_int 0)] UNSPECV_POOL_SECTION)]
""
".previous"
{
switch_to_section (current_function_section ());
return "";
}
[(set_attr "length" "0")])
(define_insn "main_base_31_small"