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

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

[Senthil Kumar Selvaraj]

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.

The code merely adds cflags to compiler_flags in the check-flags proc, 
exactly the same way as multilib_flags is added.

Regards
Senthil

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


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] "
     }

Re: [PING][PATCH,dejagnu] Allow dg-skip-if to use compiler flags specified through set_board_info cflags

[Senthil Kumar Selvaraj]

Hello

On Sat, Aug 11, 2012 at 11:09:03PM +0530, 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.
> 
> The code merely adds cflags to compiler_flags in the check-flags proc, 
> exactly the same way as multilib_flags is added.
> 
> Regards
> Senthil
> 
>     * lib/target-supports-dg.exp (check-flags): Add cflags from board
>       config to compiler_flags
> 
> 
> 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] "
>      }

Does the patch look ok?

Regards
Senthil

Re: [PATCH] Allow dg-skip-if to use compiler flags specified through set_board_info cflags

[Mike Stump]

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.

Re: [PATCH] Allow dg-skip-if to use compiler flags specified through set_board_info cflags

[Senthil Kumar Selvaraj]

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.

I forgot to mention that I don't have commit access - would you mind comitting
the patch to trunk for me?

Regards
Senthil

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

[Senthil Kumar Selvaraj]

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


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] "
     }