]> git.ipfire.org Git - thirdparty/gcc.git/commit
nios2: Support for GOT-relative DW_EH_PE_datarel encoding.
authorSandra Loosemore <sandra@codesourcery.com>
Sat, 1 Feb 2020 00:46:50 +0000 (16:46 -0800)
committerSandra Loosemore <sandra@codesourcery.com>
Sat, 1 Feb 2020 00:46:50 +0000 (16:46 -0800)
commit2d33dcfe9f0494c9b56a8d704c3d27c5a4329ebc
treef552c2ff1a82a7100904140e5c4f9f20085aee0e
parent20fa702b32c7ec7a7215df322d3e8d19b79f2068
nios2: Support for GOT-relative DW_EH_PE_datarel encoding.

On nios2-linux-gnu, there has been a long-standing bug in C++ exception
handling that sometimes resulted in link errors like

../nios2-linux-gnu/bin/ld: FDE encoding in /tmp/cccfpQ2l.o(.eh_frame) prevents .eh_frame_hdr table being created

when building some shared libraries or PIE executables.  The root of
the problem is that GCC was incorrectly emitting an absolute encoding
in EH tables for PIC.  This patch changes it to use either
DW_EH_PE_indirect (for global) or DW_EH_PE_datarel (for local), and
fixes libgcc so it can find the address of the GOT as the base address
for DW_EH_PE_datarel.

Complicating matters somewhat, GAS was missing support for
%gotoff(symbol) relocation syntax.  I have just pushed a fix for that,
but I've added a configure check to test for presence of the binutils
support and fall back to the current absolute encoding (which works
most of the time) if it is not available.  Once the fix makes it into
an official binutils release it might be appropriate to make this
error out instead.

Since this is a wrong-code bug and affects only nios2 target, I think
this is appropriate for Stage 4.  I regression-tested on both
nios2-linux-gnu and nios2-elf, with and without the binutils support
present, before committing this.

2020-01-31  Sandra Loosemore  <sandra@codesourcery.com>

gcc/
* configure.ac [nios2-*-*]: Check HAVE_AS_NIOS2_GOTOFF_RELOCATION.
* config.in: Regenerated.
* configure: Regenerated.
* config/nios2/nios2.h (ASM_PREFERRED_EH_DATA_FORMAT): Fix handling
for PIC when HAVE_AS_NIOS2_GOTOFF_RELOCATION.
(ASM_MAYBE_OUTPUT_ENCODED_ADDR_RTX): New.

gcc/testsuite/
* g++.target/nios2/hello-pie.C: New.
* g++.target/nios2/nios2.exp: New.

libgcc/
* config.host [nios2-*-linux*] (tmake_file, tm_file): Adjust.
* config/nios2-elf-lib.h: New.
* unwind-dw2-fde-dip.c (_Unwind_IteratePhdrCallback): Use existing
code for finding GOT base for nios2.
12 files changed:
gcc/ChangeLog
gcc/config.in
gcc/config/nios2/nios2.h
gcc/configure
gcc/configure.ac
gcc/testsuite/ChangeLog
gcc/testsuite/g++.target/nios2/hello-pie.C [new file with mode: 0644]
gcc/testsuite/g++.target/nios2/nios2.exp [new file with mode: 0644]
libgcc/ChangeLog
libgcc/config.host
libgcc/config/nios2/elf-lib.h [new file with mode: 0644]
libgcc/unwind-dw2-fde-dip.c