public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [Patch, microblaze, testsuite] Remove check for target_config_cflags
@ 2013-01-21  5:46 David Holsgrove
  2013-01-23 15:12 ` Michael Eager
  0 siblings, 1 reply; 2+ messages in thread
From: David Holsgrove @ 2013-01-21  5:46 UTC (permalink / raw)
  To: gcc-patches

[-- Attachment #1: Type: text/plain, Size: 423 bytes --]

Remove test for target_config_cflags for microblaze - not set
anywhere, and causes error while
running testsuite;

ERROR: tcl error sourcing
[SNIP]/gcc/gcc/testsuite/gcc.target/microblaze/microblaze.exp.
ERROR: can't read "target_config_cflags": no such variable

gcc/testsuite/Changelog:

2013-01-21  David Holsgrove <david.holsgrove@xilinx.com>

 * gcc.target/microblaze/microblaze.exp: Remove target_config_cflags check

[-- Attachment #2: 0001-Remove-obsolete-check-for-target_config_cflags-no-lo.patch --]
[-- Type: application/octet-stream, Size: 1225 bytes --]

From 6e3eaf7999818adf30d42b17754964a8e540b225 Mon Sep 17 00:00:00 2001
From: David Holsgrove <david.holsgrove@xilinx.com>
Date: Wed, 21 Nov 2012 12:31:04 +1000
Subject: [PATCH] Remove obsolete check for target_config_cflags, no longer
 set

gcc/testsuite/Changelog:

2013-01-21  David Holsgrove <david.holsgrove@xilinx.com>

 * gcc.target/microblaze/microblaze.exp: Remove target_config_cflags check

Signed-off-by: David Holsgrove <david.holsgrove@xilinx.com>
---
 gcc/testsuite/gcc.target/microblaze/microblaze.exp |    6 ------
 1 file changed, 6 deletions(-)

diff --git a/gcc/testsuite/gcc.target/microblaze/microblaze.exp b/gcc/testsuite/gcc.target/microblaze/microblaze.exp
index 621200f..7766297 100644
--- a/gcc/testsuite/gcc.target/microblaze/microblaze.exp
+++ b/gcc/testsuite/gcc.target/microblaze/microblaze.exp
@@ -22,12 +22,6 @@ if { ![istarget microblaze*-*-*] } then {
   return
 }
 
-# Don't execute when we are testing some configuration of GCC or G++.
-# This we figure out by looking at target_config_cflags which needs to be "" for microblaze.exp
-if { $target_config_cflags != "" } {
-  return 
-}
-
 global TORTURE_OPTIONS
 set TORTURE_OPTIONS [list \
                          { -O0 } \
-- 
1.7.9.5


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [Patch, microblaze, testsuite] Remove check for target_config_cflags
  2013-01-21  5:46 [Patch, microblaze, testsuite] Remove check for target_config_cflags David Holsgrove
@ 2013-01-23 15:12 ` Michael Eager
  0 siblings, 0 replies; 2+ messages in thread
From: Michael Eager @ 2013-01-23 15:12 UTC (permalink / raw)
  To: David Holsgrove; +Cc: gcc-patches

On 01/20/2013 09:46 PM, David Holsgrove wrote:
> Remove test for target_config_cflags for microblaze - not set
> anywhere, and causes error while
> running testsuite;
>
> ERROR: tcl error sourcing
> [SNIP]/gcc/gcc/testsuite/gcc.target/microblaze/microblaze.exp.
> ERROR: can't read "target_config_cflags": no such variable
>
> gcc/testsuite/Changelog:
>
> 2013-01-21  David Holsgrove <david.holsgrove@xilinx.com>
>
>   * gcc.target/microblaze/microblaze.exp: Remove target_config_cflags check

Committed.


-- 
Michael Eager	 eager@eagercon.com
1960 Park Blvd., Palo Alto, CA 94306  650-325-8077


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2013-01-23 15:12 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-01-21  5:46 [Patch, microblaze, testsuite] Remove check for target_config_cflags David Holsgrove
2013-01-23 15:12 ` Michael Eager

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).