From b60d63cbaee16a95790c09dd6c23e346639ee180 Mon Sep 17 00:00:00 2001 From: Jiong Wang Date: Thu, 6 Aug 2015 15:57:36 +0000 Subject: [PATCH] [AArch64] Tighten direct call pattern to repair -fno-plt 2015-08-06 Jiong Wang gcc/ * config/aarch64/aarch64-protos.h (aarch64_is_noplt_call_p): New declaration. * config/aarch64/aarch64.c (aarch64_is_noplt_call_p): New function. * config/aarch64/aarch64.md (call_value_symbol): Check noplt scenarios. (call_symbol): Likewise. gcc/testsuite/ * gcc.target/aarch64/noplt_1.c: New testcase. * gcc.target/aarch64/noplt_2.c: Likewise. From-SVN: r226681 --- gcc/ChangeLog | 7 +++++++ gcc/config/aarch64/aarch64-protos.h | 1 + gcc/config/aarch64/aarch64.c | 18 ++++++++++++++++++ gcc/config/aarch64/aarch64.md | 6 ++++-- gcc/testsuite/ChangeLog | 5 +++++ gcc/testsuite/gcc.target/aarch64/noplt_1.c | 15 +++++++++++++++ gcc/testsuite/gcc.target/aarch64/noplt_2.c | 18 ++++++++++++++++++ 7 files changed, 68 insertions(+), 2 deletions(-) create mode 100644 gcc/testsuite/gcc.target/aarch64/noplt_1.c create mode 100644 gcc/testsuite/gcc.target/aarch64/noplt_2.c diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 43df172a4db..2b364ce83b9 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,10 @@ +2015-08-06 Jiong Wang + + * config/aarch64/aarch64-protos.h (aarch64_is_noplt_call_p): New declaration. + * config/aarch64/aarch64.c (aarch64_is_noplt_call_p): New function. + * config/aarch64/aarch64.md (call_value_symbol): Check noplt scenarios. + (call_symbol): Likewise. + 2015-08-06 Venkataramanan Kumar * tree-vect-patterns.c (vect_recog_mult_pattern): New function diff --git a/gcc/config/aarch64/aarch64-protos.h b/gcc/config/aarch64/aarch64-protos.h index 5d8902ff327..32b5d0958a6 100644 --- a/gcc/config/aarch64/aarch64-protos.h +++ b/gcc/config/aarch64/aarch64-protos.h @@ -258,6 +258,7 @@ bool aarch64_handle_option (struct gcc_options *, struct gcc_options *, const struct cl_decoded_option *, location_t); bool aarch64_is_extend_from_extract (machine_mode, rtx, rtx); bool aarch64_is_long_call_p (rtx); +bool aarch64_is_noplt_call_p (rtx); bool aarch64_label_mentioned_p (rtx); void aarch64_declare_function_name (FILE *, const char*, tree); bool aarch64_legitimate_pic_operand_p (rtx); diff --git a/gcc/config/aarch64/aarch64.c b/gcc/config/aarch64/aarch64.c index 1394ed7decc..e991a494cfc 100644 --- a/gcc/config/aarch64/aarch64.c +++ b/gcc/config/aarch64/aarch64.c @@ -744,6 +744,24 @@ aarch64_is_long_call_p (rtx sym) return aarch64_decl_is_long_call_p (SYMBOL_REF_DECL (sym)); } +/* Return true if calls to symbol-ref SYM should not go through + plt stubs. */ + +bool +aarch64_is_noplt_call_p (rtx sym) +{ + const_tree decl = SYMBOL_REF_DECL (sym); + + if (flag_pic + && decl + && (!flag_plt + || lookup_attribute ("noplt", DECL_ATTRIBUTES (decl))) + && !targetm.binds_local_p (decl)) + return true; + + return false; +} + /* Return true if the offsets to a zero/sign-extract operation represent an expression that matches an extend operation. The operands represent the paramters from diff --git a/gcc/config/aarch64/aarch64.md b/gcc/config/aarch64/aarch64.md index b7b04c4cbff..7f997532895 100644 --- a/gcc/config/aarch64/aarch64.md +++ b/gcc/config/aarch64/aarch64.md @@ -603,7 +603,8 @@ (use (match_operand 2 "" "")) (clobber (reg:DI LR_REGNUM))] "GET_CODE (operands[0]) == SYMBOL_REF - && !aarch64_is_long_call_p (operands[0])" + && !aarch64_is_long_call_p (operands[0]) + && !aarch64_is_noplt_call_p (operands[0])" "bl\\t%a0" [(set_attr "type" "call")] ) @@ -665,7 +666,8 @@ (use (match_operand 3 "" "")) (clobber (reg:DI LR_REGNUM))] "GET_CODE (operands[1]) == SYMBOL_REF - && !aarch64_is_long_call_p (operands[1])" + && !aarch64_is_long_call_p (operands[1]) + && !aarch64_is_noplt_call_p (operands[1])" "bl\\t%a1" [(set_attr "type" "call")] ) diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 76afd8e89fe..fb3bf07f58f 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2015-08-06 Jiong Wang + + * gcc.target/aarch64/noplt_1.c: New testcase. + * gcc.target/aarch64/noplt_2.c: Likewise. + 2015-08-06 Venkataramanan Kumar * gcc.dg/vect/vect-mult-pattern-1.c: New test. diff --git a/gcc/testsuite/gcc.target/aarch64/noplt_1.c b/gcc/testsuite/gcc.target/aarch64/noplt_1.c new file mode 100644 index 00000000000..4e9bb62ba1c --- /dev/null +++ b/gcc/testsuite/gcc.target/aarch64/noplt_1.c @@ -0,0 +1,15 @@ +/* { dg-do compile } */ +/* { dg-options "-O2 -fpic -fno-plt" } */ +/* { dg-skip-if "-mcmodel=large, no support for -fpic" { aarch64-*-* } { "-mcmodel=large" } { "" } } */ + +int* bar (void) ; + +int +foo (int a) +{ + int *b = bar (); + return b[a]; +} + +/* { dg-final { scan-assembler "#:got:" { target { aarch64_tiny || aarch64_small } } } } */ +/* { dg-final { scan-assembler "#:got_lo12:" { target aarch64_small } } } */ diff --git a/gcc/testsuite/gcc.target/aarch64/noplt_2.c b/gcc/testsuite/gcc.target/aarch64/noplt_2.c new file mode 100644 index 00000000000..718999b3c54 --- /dev/null +++ b/gcc/testsuite/gcc.target/aarch64/noplt_2.c @@ -0,0 +1,18 @@ +/* { dg-do compile } */ +/* { dg-options "-O2 -fpic" } */ +/* { dg-skip-if "-mcmodel=large, no support for -fpic" { aarch64-*-* } { "-mcmodel=large" } { "" } } */ + +__attribute__ ((noplt)) +int* bar0 (void) ; +int* bar1 (void) ; + +int +foo (int a) +{ + int *b0 = bar0 (); + int *b1 = bar1 (); + return b0[a] + b1[a]; +} + +/* { dg-final { scan-assembler-times "#:got:" 1 { target { aarch64_tiny || aarch64_small } } } } */ +/* { dg-final { scan-assembler-times "#:got_lo12:" 1 { target aarch64_small } } } */ -- 2.30.2