Patchwork [Ping] Allow dg-skip-if to use compiler flags specified through set_board_info cflags

login
register
mail settings
Submitter Senthil Kumar Selvaraj
Date Sept. 18, 2012, 6:50 a.m.
Message ID <20120918065031.GA1834@atmel.com>
Download mbox | patch
Permalink /patch/184628/
State New
Headers show

Comments

Senthil Kumar Selvaraj - Sept. 18, 2012, 6:50 a.m.
Hello,

Could one of the maintainers please check in the patch below?

It was already approved at
http://gcc.gnu.org/ml/gcc-patches/2012-08/msg00689.html

Regards
Senthil

On Tue, Aug 21, 2012 at 03:08:43PM -0700, Mike Stump wrote:
> On Aug 11, 2012, at 10:39 AM, Senthil Kumar Selvaraj wrote:
> > This patch allows cflags set in board config files using 
> > "set_board_info cflags" to be used in the selectors of
> > dg-skip-if and other dejagnu commands that use the check-flags
> > proc.
> 
> Ok.


    * lib/target-supports-dg.exp (check-flags): Add cflags from board
      config to compiler_flags

Patch

diff --git a/gcc/testsuite/lib/target-supports-dg.exp b/gcc/testsuite/lib/target-supports-dg.exp
index 2f6c4c2..bdf7476 100644
--- a/gcc/testsuite/lib/target-supports-dg.exp
+++ b/gcc/testsuite/lib/target-supports-dg.exp
@@ -304,6 +304,9 @@  proc check-flags { args } {
     # If running a subset of the test suite, $TEST_ALWAYS_FLAGS may not exist.
     catch {append compiler_flags " $TEST_ALWAYS_FLAGS "}
     set dest [target_info name]
+    if [board_info $dest exists cflags] {
+    append compiler_flags "[board_info $dest cflags] "
+    }
     if [board_info $dest exists multilib_flags] {
    append compiler_flags "[board_info $dest multilib_flags] "
     }