diff options
| author | H.J. Lu <hjl.tools@gmail.com> | 2015-02-26 17:53:48 +1100 |
|---|---|---|
| committer | Ben Elliston <bje@gnu.org> | 2015-02-26 17:53:48 +1100 |
| commit | 5256bd82343000c76bc0e48139003f90b6184347 (patch) | |
| tree | db550b08e4671a66398c84177e35d9ad4568d55d | |
| parent | 3249854999143f1a0b728e812b26e1665ebfdb4b (diff) | |
| download | dejagnu-5256bd82343000c76bc0e48139003f90b6184347.zip dejagnu-5256bd82343000c76bc0e48139003f90b6184347.tar.gz dejagnu-5256bd82343000c76bc0e48139003f90b6184347.tar.bz2 | |
* lib/target.exp (default_target_compile): Prepend multilib_flags,
instead of appending it.
Some GCC testcases need explicit GCC options to properly run. For
example gcc.target/i386/pr32219-1.c has -fpie specified explicitly:
/* { dg-options "-O2 -fpie" } */
But with multlib, eg:
make check-gcc RUNTESTFLAGS="--target_board='unix{-fpic}'"
-fpic is appended to the command line options, which overrides the command
line options specified by dg-options. multlib flags should be placed at
the beginning of the command line options, not at the end. This patch
updates default_target_compile to prepend multilib_flags, instead of
appending it.
Signed-off-by: Ben Elliston <bje@gnu.org>
| -rw-r--r-- | ChangeLog | 5 | ||||
| -rw-r--r-- | lib/target.exp | 2 |
2 files changed, 6 insertions, 1 deletions
@@ -1,3 +1,8 @@ +2015-02-26 H.J. Lu <hjl.tools@gmail.com> + + * lib/target.exp (default_target_compile): Prepend multilib_flags, + instead of appending it. + 2015-02-18 Ben Elliston <bje@gnu.org> * lib/framework.exp (log_and_exit): Set tool to "testrun" if diff --git a/lib/target.exp b/lib/target.exp index 71a78c7..30e4585 100644 --- a/lib/target.exp +++ b/lib/target.exp @@ -577,7 +577,7 @@ proc default_target_compile {source destfile type options} { } if {[board_info $dest exists multilib_flags]} { - append add_flags " [board_info $dest multilib_flags]" + set add_flags "[board_info $dest multilib_flags] $add_flags" } verbose "doing compile" |