Properly handle -fno-plt in ix86_expand_callheads/hjl/pr67215/gcc-5-branch
commit19894c6669dd0940beda4bfc5aff241883d7c740
authorH.J. Lu <hjl.tools@gmail.com>
Sun, 16 Aug 2015 11:46:20 +0000 (16 04:46 -0700)
committerH.J. Lu <hjl.tools@gmail.com>
Tue, 18 Aug 2015 11:49:35 +0000 (18 04:49 -0700)
tree23de17e52d43de736176e27279a37ec5d59f1909
parent2381bc33b7bcc796f094cfc28b7f20a09b74d088
Properly handle -fno-plt in ix86_expand_call

prepare_call_address in calls.c is the wrong place to handle -fno-plt.
We shoudn't force function address into register and hope that load
function address via GOT and indirect call via register will be folded
into indirect call via GOT, which doesn't always happen.  Also non-PIC
case can only be handled in backend.  Instead, backend should expand
external function call into indirect call via GOT for -fno-plt.

This patch reverts -fno-plt in prepare_call_address and handles it in
ix86_expand_call.  Other backends may need similar changes to support
-fno-plt.  Alternately, we can introduce a target hook to indicate
whether an external function should be called via register for -fno-plt
so that i386 backend can disable it in prepare_call_address.

gcc/

PR target/67215
* calls.c (prepare_call_address): Don't handle -fno-plt here.
* config/i386/i386.c (ix86_expand_call): Generate indirect call
via GOT for -fno-plt.  Support indirect call via GOT for x32.
* config/i386/predicates.md (sibcall_memory_operand): Allow
GOT memory operand.

gcc/testsuite/

PR target/67215
* gcc.target/i386/pr67215-1.c: New test.
* gcc.target/i386/pr67215-2.c: Likewise.
* gcc.target/i386/pr67215-3.c: Likewise.
gcc/calls.c
gcc/config/i386/i386.c
gcc/config/i386/predicates.md
gcc/testsuite/gcc.target/i386/pr67215-1.c [new file with mode: 0644]
gcc/testsuite/gcc.target/i386/pr67215-2.c [new file with mode: 0644]
gcc/testsuite/gcc.target/i386/pr67215-3.c [new file with mode: 0644]