Set DF_1_PIE in gld${EMULATION_NAME}_after_parse

We can't add OPTION_PIE to gld${EMULATION_NAME}_handle_option since
it has been handled in parse_args in lexsup.c.  This patch moves
setting DF_1_PIE to gld${EMULATION_NAME}_after_parse.

ld/

	* emultempl/alphaelf.em (alpha_after_parse): Call
	gld${EMULATION_NAME}_after_parse instead of
	after_parse_default.
	* emultempl/cr16elf.em (cr16elf_after_parse): Likewise.
	* emultempl/crxelf.em (crxelf_after_parse); Likewise.
	* emultempl/hppaelf.em (hppaelf_after_parse): Likewise.
	* emultempl/mipself.em (mips_after_parse): Likewise.
	* emultempl/nds32elf.em (nds32_elf_after_parse): Likewise.
	* emultempl/elf32.em: Don't include ldlex.h.
	(gld${EMULATION_NAME}_after_parse): New function.
	(gld${EMULATION_NAME}_handle_option) [GENERATE_PIE_SCRIPT]
	<OPTION_PIE>: Removed.
	(ld_${EMULATION_NAME}_emulation): Replace after_parse_default
	with gld${EMULATION_NAME}_after_parse.
	* emultempl/ia64elf.em (gld${EMULATION_NAME}_after_parse):
	Renamed to ...
	(ia64elf_after_parse): This.  Call
	gld${EMULATION_NAME}_after_parse instead of after_parse_default.
	(LDEMUL_AFTER_PARSE): Replace gld${EMULATION_NAME}_after_parse
	with ia64elf_after_parse.

ld/testsuite/

	* ld-elf/pie.d: New test.

diff --git a/ld/ChangeLog b/ld/ChangeLog
index 88a46c5fce5f4bc15341de0ab9cd75d22d9d2c62..93a852a6de0f50d7e1aba38377e8dcfbbc7a9347 100644 (file)
--- a/ld/ChangeLog
+++ b/ld/ChangeLog
@@ -1,3 +1,26 @@
+2015-09-22  H.J. Lu  <hongjiu.lu@intel.com>
+
+       * emultempl/alphaelf.em (alpha_after_parse): Call
+       gld${EMULATION_NAME}_after_parse instead of
+       after_parse_default.
+       * emultempl/cr16elf.em (cr16elf_after_parse): Likewise.
+       * emultempl/crxelf.em (crxelf_after_parse); Likewise.
+       * emultempl/hppaelf.em (hppaelf_after_parse): Likewise.
+       * emultempl/mipself.em (mips_after_parse): Likewise.
+       * emultempl/nds32elf.em (nds32_elf_after_parse): Likewise.
+       * emultempl/elf32.em: Don't include ldlex.h.
+       (gld${EMULATION_NAME}_after_parse): New function.
+       (gld${EMULATION_NAME}_handle_option) [GENERATE_PIE_SCRIPT]
+       <OPTION_PIE>: Removed.
+       (ld_${EMULATION_NAME}_emulation): Replace after_parse_default
+       with gld${EMULATION_NAME}_after_parse.
+       * emultempl/ia64elf.em (gld${EMULATION_NAME}_after_parse):
+       Renamed to ...
+       (ia64elf_after_parse): This.  Call
+       gld${EMULATION_NAME}_after_parse instead of after_parse_default.
+       (LDEMUL_AFTER_PARSE): Replace gld${EMULATION_NAME}_after_parse
+       with ia64elf_after_parse.
+
 2015-09-22  Rainer Orth  <ro@CeBiTec.Uni-Bielefeld.DE>
 
        * emulparams/solaris2.sh (GENERATE_PIE_SCRIPT): Set to yes.

diff --git a/ld/emultempl/alphaelf.em b/ld/emultempl/alphaelf.em
index 1301c6f004390b5e53ef3394ed0cc362a6aacf37..1db94c7f00ffad4748e88dc671714c0d018f093e 100644 (file)
--- a/ld/emultempl/alphaelf.em
+++ b/ld/emultempl/alphaelf.em
@@ -82,7 +82,7 @@ alpha_after_parse (void)
                                   exp_nameop (SIZEOF_HEADERS, NULL)),
                        NULL);
 
-  after_parse_default ();
+  gld${EMULATION_NAME}_after_parse ();
 }
 
 static void

diff --git a/ld/emultempl/cr16elf.em b/ld/emultempl/cr16elf.em
index 276dce4b8339f6018516656223ba1df43ca34c07..48dcf14045b2a7153a728eab52137e30510b8da4 100644 (file)
--- a/ld/emultempl/cr16elf.em
+++ b/ld/emultempl/cr16elf.em
@@ -119,7 +119,7 @@ cr16elf_after_parse (void)
      is true the link sometimes fails.  */
   config.magic_demand_paged = FALSE;
 
-  after_parse_default ();
+  gld${EMULATION_NAME}_after_parse ();
 }
 
 /* This is called after the sections have been attached to output

diff --git a/ld/emultempl/crxelf.em b/ld/emultempl/crxelf.em
index 8fefb82d394beeb60700b811c2eb7ed794f52cdc..172aa9b516fb135458fd7e746a7d9170629b5650 100644 (file)
--- a/ld/emultempl/crxelf.em
+++ b/ld/emultempl/crxelf.em
@@ -42,7 +42,7 @@ crxelf_after_parse (void)
      is true the link sometimes fails.  */
   config.magic_demand_paged = FALSE;
 
-  after_parse_default ();
+  gld${EMULATION_NAME}_after_parse ();
 }
 
 /* This is called after the sections have been attached to output

diff --git a/ld/emultempl/elf32.em b/ld/emultempl/elf32.em
index 5fe576ea48063bb326eb39a5d7b8a8a49136e18d..b7ddca38062ee50c667697a126f4de3e46f31379 100644 (file)
--- a/ld/emultempl/elf32.em
+++ b/ld/emultempl/elf32.em
@@ -49,7 +49,6 @@ fragment <<EOF
 #include "ldmain.h"
 #include "ldmisc.h"
 #include "ldexp.h"
-#include "ldlex.h"
 #include "ldlang.h"
 #include "ldfile.h"
 #include "ldemul.h"
@@ -61,6 +60,7 @@ fragment <<EOF
 
 /* Declare functions used by various EXTRA_EM_FILEs.  */
 static void gld${EMULATION_NAME}_before_parse (void);
+static void gld${EMULATION_NAME}_after_parse (void);
 static void gld${EMULATION_NAME}_after_open (void);
 static void gld${EMULATION_NAME}_before_allocation (void);
 static void gld${EMULATION_NAME}_after_allocation (void);
@@ -108,6 +108,21 @@ gld${EMULATION_NAME}_before_parse (void)
 EOF
 fi
 
+if test x"$LDEMUL_AFTER_PARSE" != xgld"$EMULATION_NAME"_after_parse; then
+fragment <<EOF
+
+static void
+gld${EMULATION_NAME}_after_parse (void)
+{
+  if (bfd_link_pie (&link_info))
+    link_info.flags_1 |= (bfd_vma) DF_1_PIE;
+
+  after_parse_default ();
+}
+
+EOF
+fi
+
 if test x"$LDEMUL_RECOGNIZED_FILE" != xgld"${EMULATION_NAME}"_load_symbols; then
 fragment <<EOF
 /* Handle the generation of DT_NEEDED tags.  */
@@ -2290,13 +2305,6 @@ fragment <<EOF
 
 EOF
 fi
-if test x"$GENERATE_PIE_SCRIPT" = xyes; then
-fragment <<EOF
-    case OPTION_PIE:
-      link_info.flags_1 |= (bfd_vma) DF_1_PIE;
-      break;
-EOF
-fi
 fragment <<EOF
     case 'z':
       if (strcmp (optarg, "defs") == 0)
@@ -2459,7 +2467,7 @@ struct ld_emulation_xfer_struct ld_${EMULATION_NAME}_emulation =
   ${LDEMUL_BEFORE_PARSE-gld${EMULATION_NAME}_before_parse},
   ${LDEMUL_SYSLIB-syslib_default},
   ${LDEMUL_HLL-hll_default},
-  ${LDEMUL_AFTER_PARSE-after_parse_default},
+  ${LDEMUL_AFTER_PARSE-gld${EMULATION_NAME}_after_parse},
   ${LDEMUL_AFTER_OPEN-gld${EMULATION_NAME}_after_open},
   ${LDEMUL_AFTER_ALLOCATION-gld${EMULATION_NAME}_after_allocation},
   ${LDEMUL_SET_OUTPUT_ARCH-set_output_arch_default},

diff --git a/ld/emultempl/hppaelf.em b/ld/emultempl/hppaelf.em
index 26ccc0fba7dd0c4d8ab9f57727f9cd8d9e98169a..08071496f917cb76a1c5914e9c6694d625d7e2b5 100644 (file)
--- a/ld/emultempl/hppaelf.em
+++ b/ld/emultempl/hppaelf.em
@@ -59,7 +59,7 @@ hppaelf_after_parse (void)
                          NULL);
   */
 
-  after_parse_default ();
+  gld${EMULATION_NAME}_after_parse ();
 }
 
 /* This is called before the input files are opened.  We create a new

diff --git a/ld/emultempl/ia64elf.em b/ld/emultempl/ia64elf.em
index 4de45e844e7f95bff4dab7fcd9e35943a4b1bcc8..ddfa4b570f0010d0c1a278be00917a5f4dc4aece 100644 (file)
--- a/ld/emultempl/ia64elf.em
+++ b/ld/emultempl/ia64elf.em
@@ -31,12 +31,12 @@ fragment <<EOF
 static int itanium = 0;
 
 static void
-gld${EMULATION_NAME}_after_parse (void)
+ia64elf_after_parse (void)
 {
   link_info.relax_pass = 2;
   bfd_elf${ELFSIZE}_ia64_after_parse (itanium);
 
-  after_parse_default ();
+  gld${EMULATION_NAME}_after_parse ();
 }
 
 EOF
@@ -61,5 +61,5 @@ PARSE_AND_LIST_ARGS_CASES='
       break;
 '
 
-LDEMUL_AFTER_PARSE=gld${EMULATION_NAME}_after_parse
+LDEMUL_AFTER_PARSE=ia64elf_after_parse
 source_em ${srcdir}/emultempl/needrelax.em

diff --git a/ld/emultempl/mipself.em b/ld/emultempl/mipself.em
index 0eb0813b4e4edc9a7429658137b63959c2ec9301..1c60c66d61e94cca76b4ef6ae88f8ebb5ba187f0 100644 (file)
--- a/ld/emultempl/mipself.em
+++ b/ld/emultempl/mipself.em
@@ -47,7 +47,7 @@ mips_after_parse (void)
       link_info.emit_hash = TRUE;
       link_info.emit_gnu_hash = FALSE;
     }
-  after_parse_default ();
+  gld${EMULATION_NAME}_after_parse ();
 }
 
 struct hook_stub_info

diff --git a/ld/emultempl/nds32elf.em b/ld/emultempl/nds32elf.em
index e74652aa53ed60e91351e93c5237df8a14f54713..73a49f7cf58dc2b1ac8e9316946a46398c5816e0 100644 (file)
--- a/ld/emultempl/nds32elf.em
+++ b/ld/emultempl/nds32elf.em
@@ -94,7 +94,7 @@ nds32_elf_after_parse (void)
       target_optimize = target_optimize & (!NDS32_RELAX_EX9_ON);
     }
 
-  after_parse_default ();
+  gld${EMULATION_NAME}_after_parse ();
 }
 
 static void

diff --git a/ld/testsuite/ChangeLog b/ld/testsuite/ChangeLog
index 575d0482c85ddb05b92efd9032383c288bc0fa18..78de2b29ccde58927a8d615bacf4e256546c6872 100644 (file)
--- a/ld/testsuite/ChangeLog
+++ b/ld/testsuite/ChangeLog
@@ -1,3 +1,7 @@
+2015-09-22  H.J. Lu  <hongjiu.lu@intel.com>
+
+       * ld-elf/pie.d: New test.
+
 2015-09-18  Alan Modra  <amodra@gmail.com>
 
        PR ld/18963

diff --git a/ld/testsuite/ld-elf/pie.d b/ld/testsuite/ld-elf/pie.d
new file mode 100644 (file)
index 0000000..d68b962
--- /dev/null
+++ b/ld/testsuite/ld-elf/pie.d
@@ -0,0 +1,8 @@
+#source: start.s
+#readelf: -d -W
+#ld: -pie
+#target: *-*-linux* *-*-gnu* *-*-solaris*
+
+#...
+ +0x[0-9a-f]+ +\(FLAGS_1\) +Flags: +PIE
+#pass