Another altivec testsuite patch

Another altivec testsuite patch

[Paul Gilliam]

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

The gcc compiler needs -maltivec to enable altivec support.

OK to commit?  Is this an obvious patch I should have just committed without 
asking?

-=# Paul Gilliam #=-


[-- Attachment #2: altivec2.patch --]
[-- Type: text/x-diff, Size: 1964 bytes --]

diff -Naur testsuite.old/ChangeLog testsuite/ChangeLog
--- testsuite.old/ChangeLog	2005-03-02 13:28:23.757211464 -0800
+++ testsuite/ChangeLog	2005-03-02 13:56:01.659188904 -0800
@@ -1,3 +1,8 @@
+2005-03-02  Paul Gilliam  <pgilliam@us.ibm.com>
+
+        * gdb.arch/altivec-abi.exp: Added -maltivec to compiler flags.
+        * gdb.arch/altivec-regs.exp: Likewise
+
 2004-03-01  Paul Gilliam  <pgilliam@us.ibm.com>
 
         * lib/gdb.exp: added 'skip_altivec_tests', a modification of
diff -Naur testsuite.old/gdb.arch/altivec-abi.exp testsuite/gdb.arch/altivec-abi.exp
--- testsuite.old/gdb.arch/altivec-abi.exp	2005-03-02 13:28:23.759211160 -0800
+++ testsuite/gdb.arch/altivec-abi.exp	2005-03-02 13:30:27.210216176 -0800
@@ -41,7 +41,9 @@
 set binfile ${objdir}/${subdir}/${testfile}
 set srcfile ${testfile}.c
 
-if  { [gdb_compile ${srcdir}/${subdir}/${srcfile} ${binfile} executable {debug additional_flags=-w}] != "" } {
+set compile_options {debug additional_flags=-w additional_flags=-maltivec}
+
+if  { [gdb_compile ${srcdir}/${subdir}/${srcfile} ${binfile} executable $compile_options] != "" } {
      gdb_suppress_entire_file "Testcase compile failed, so all tests in this file will automatically fail."
 }
 
diff -Naur testsuite.old/gdb.arch/altivec-regs.exp testsuite/gdb.arch/altivec-regs.exp
--- testsuite.old/gdb.arch/altivec-regs.exp	2005-03-02 13:28:23.759211160 -0800
+++ testsuite/gdb.arch/altivec-regs.exp	2005-03-02 13:30:27.211216024 -0800
@@ -41,7 +41,9 @@
 set binfile ${objdir}/${subdir}/${testfile}
 set srcfile ${testfile}.c
 
-if  { [gdb_compile ${srcdir}/${subdir}/${srcfile} ${binfile} executable {debug additional_flags=-w}] != "" } {
+set compile_options {debug additional_flags=-w additional_flags=-maltivec}
+
+if  { [gdb_compile ${srcdir}/${subdir}/${srcfile} ${binfile} executable $compile_options] != "" } {
      gdb_suppress_entire_file "Testcase compile failed, so all tests in this file will automatically fail."
 }
 

Re: Another altivec testsuite patch

[Daniel Jacobowitz]

On Wed, Mar 02, 2005 at 01:53:50PM -0800, Paul Gilliam wrote:
> The gcc compiler needs -maltivec to enable altivec support.
> 
> OK to commit?  Is this an obvious patch I should have just committed without 
> asking?

Same comments as the other patch: ChangeLog formatting, and GCC
specific options.


-- 
Daniel Jacobowitz
CodeSourcery, LLC