This is the mail archive of the
binutils@sources.redhat.com
mailing list for the binutils project.
[PATCH] Enable --relax option by default for CRX
- From: "Tomer Levi" <Tomer dot Levi at nsc dot com>
- To: binutils at sources dot redhat dot com
- Date: Tue, 5 Oct 2004 13:32:11 +0200
- Subject: [PATCH] Enable --relax option by default for CRX
Hi,
The following patch enables the linker's --relax option by default for CRX
(and provides the user a --no-relax to disable it).
Checked and tested.
o.k. to apply ?
2004-10-05 Tomer Levi <Tomer.Levi@nsc.com>
* emultempl/crxelf.em (disable_relaxation): Add.
(crxelf_before_allocation): Enable --relax option by
default.
(PARSE_AND_LIST_PROLOGUE): Add OPTION_NO_RELAX.
(PARSE_AND_LIST_LONGOPTS, PARSE_AND_LIST_OPTIONS): Add
--no-relax.
(PARSE_AND_LIST_ARGS_CASES): Handle OPTION_NO_RELAX.
--- crxelf.em 7 Jul 2004 17:28:52 -0000 1.1
+++ crxelf.em 5 Oct 2004 11:20:26 -0000
@@ -26,6 +26,9 @@ cat >>e${EMULATION_NAME}.c <<EOF
#include "ldctor.h"
+/* Flag for the emulation-specific "--no-relax" option. */
+static bfd_boolean disable_relaxation = FALSE;
+
static void crxelf_after_parse (void);
static void
@@ -38,13 +41,54 @@ crxelf_after_parse (void)
config.sort_common = TRUE;
/* Don't create a demand-paged executable, since this feature isn't
- meaninful in CR16C embedded systems. Moreover, when
magic_demand_paged
+ meaninful in CRX embedded systems. Moreover, when magic_demand_paged
is true the link sometimes fails. */
config.magic_demand_paged = FALSE;
}
+/* This is called after the sections have been attached to output
+ sections, but before any sizes or addresses have been set. */
+
+static void
+crxelf_before_allocation (void)
+{
+ /* Call the default first. */
+ gld${EMULATION_NAME}_before_allocation ();
+
+ /* Enable relaxation by default if the "--no-relax" option was not
+ specified. This is done here instead of in the before_parse hook
+ because there is a check in main() to prohibit use of --relax and
+ -r together. */
+
+ if (!disable_relaxation)
+ command_line.relax = TRUE;
+}
+
EOF
+# Define some shell vars to insert bits of code into the standard elf
+# parse_args and list_options functions.
+#
+PARSE_AND_LIST_PROLOGUE='
+#define OPTION_NO_RELAX 301
+'
+
+PARSE_AND_LIST_LONGOPTS='
+ { "no-relax", no_argument, NULL, OPTION_NO_RELAX},
+'
+
+PARSE_AND_LIST_OPTIONS='
+ fprintf (file, _(" --no-relax Do not relax
branches\n"));
+'
+
+PARSE_AND_LIST_ARGS_CASES='
+ case OPTION_NO_RELAX:
+ disable_relaxation = TRUE;
+ break;
+'
+
# Put these extra crx-elf routines in ld_${EMULATION_NAME}_emulation
#
LDEMUL_AFTER_PARSE=crxelf_after_parse
+LDEMUL_BEFORE_ALLOCATION=crxelf_before_allocation
+
Regards,
Tomer Levi
Software Engineer phone:
+49-8141-35-1378
CompactRISC Development Tools fax: +49-8141-35-11-1378
National Semiconductor GmbH mailto:Tomer.Levi@nsc.com
Livry-Gargan Str. 10 internet:
http://www.national.com
82256 Fuerstenfeldbruck, Germany