This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [Jim Blandy <jimb@redhat.com>] RFA: testsuite: pass optimizationflag in the proper way
- From: Fernando Nasser <fnasser at redhat dot com>
- To: Jim Blandy <jimb at redhat dot com>
- Cc: gdb-patches at sources dot redhat dot com
- Date: Sun, 22 Dec 2002 06:18:26 -0500
- Subject: Re: [Jim Blandy <jimb@redhat.com>] RFA: testsuite: pass optimizationflag in the proper way
- Organization: Red Hat Canada
- References: <vt27ke4pmde.fsf@zenia.red-bean.com>
> ! # Why do we need to specify '-O1' here? Shouldn't that sort of
> ! # compilation parameter be under the control of the test framework
> ! # (the target_list and so on)?
Because you would be changing it for all tests while this test seems to
require a specific setting.for some reason.
This part is approved:
> ! if { [gdb_compile "$srcdir/$subdir/$srcfile" "$binfile" \
> ! executable {debug additional_flags=-w optimize=-O1}] != "" } {
The comment could be:
"Why do we need this specific optimization level here?"
or the answer to this question if you know why.
Sorry for the delay.
Regards,
Fernando
Jim Blandy wrote:
Ping on this test suite patch.
------------------------------------------------------------------------
Subject:
RFA: testsuite: pass optimization flag in the proper way
From:
Jim Blandy <jimb@redhat.com>
Date:
18 Nov 2002 22:38:55 -0500
To:
gdb-patches@sources.redhat.com
2002-11-18 Jim Blandy <jimb@redhat.com>
* gdb.trace/tfind.exp: Pass '-O1' flag via the 'optimize' option,
rather than as part of the output file name.
Index: gdb/testsuite/gdb.trace/tfind.exp
===================================================================
RCS file: /cvs/src/src/gdb/testsuite/gdb.trace/tfind.exp,v
retrieving revision 1.2
diff -c -r1.2 tfind.exp
*** gdb/testsuite/gdb.trace/tfind.exp 6 Mar 2001 08:22:00 -0000 1.2
--- gdb/testsuite/gdb.trace/tfind.exp 19 Nov 2002 02:48:02 -0000
***************
*** 53,60 ****
set testfile "actions"
set srcfile ${testfile}.c
set binfile $objdir/$subdir/$testfile
! if { [gdb_compile "$srcdir/$subdir/$srcfile" "$binfile -O1" \
! executable {debug additional_flags=-w}] != "" } {
gdb_suppress_entire_file "Testcase compile failed, so all tests in this file will automatically fail."
}
gdb_load $binfile
--- 53,64 ----
set testfile "actions"
set srcfile ${testfile}.c
set binfile $objdir/$subdir/$testfile
!
! # Why do we need to specify '-O1' here? Shouldn't that sort of
! # compilation parameter be under the control of the test framework
! # (the target_list and so on)?
! if { [gdb_compile "$srcdir/$subdir/$srcfile" "$binfile" \
! executable {debug additional_flags=-w optimize=-O1}] != "" } {
gdb_suppress_entire_file "Testcase compile failed, so all tests in this file will automatically fail."
}
gdb_load $binfile
--
Fernando Nasser
Red Hat Canada Ltd. E-Mail: fnasser@redhat.com
2323 Yonge Street, Suite #300
Toronto, Ontario M4P 2C9