gdb.trace/tfile-avx.c: Change ymm15 to xmm15 for old gcc.

Message ID 1455306432-7237-1-git-send-email-koriakin@0x04.net
State New, archived
Headers

Commit Message

Marcin Kościelnicki Feb. 12, 2016, 7:47 p.m. UTC
  gcc older than 4.9 doesn't understand ymm15 as a register name.  Use
xmm15 instead.

gdb/testsuite/ChangeLog:

	* gdb.trace/tfile-avx.c (main): Change ymm15 to xmm15.
---
OK to push?

 gdb/testsuite/ChangeLog             | 4 ++++
 gdb/testsuite/gdb.trace/tfile-avx.c | 4 +++-
 2 files changed, 7 insertions(+), 1 deletion(-)
  

Comments

Pedro Alves Feb. 12, 2016, 7:57 p.m. UTC | #1
On 02/12/2016 07:47 PM, Marcin Kościelnicki wrote:
> gcc older than 4.9 doesn't understand ymm15 as a register name.  Use
> xmm15 instead.
> 
> gdb/testsuite/ChangeLog:
> 
> 	* gdb.trace/tfile-avx.c (main): Change ymm15 to xmm15.
> ---
> OK to push?
> 

OK.

Thanks,
Pedro Alves
  
Marcin Kościelnicki Feb. 12, 2016, 9:34 p.m. UTC | #2
On 12/02/16 20:57, Pedro Alves wrote:
> On 02/12/2016 07:47 PM, Marcin Kościelnicki wrote:
>> gcc older than 4.9 doesn't understand ymm15 as a register name.  Use
>> xmm15 instead.
>>
>> gdb/testsuite/ChangeLog:
>>
>> 	* gdb.trace/tfile-avx.c (main): Change ymm15 to xmm15.
>> ---
>> OK to push?
>>
>
> OK.
>
> Thanks,
> Pedro Alves
>

Thanks, pushed.
  

Patch

diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog
index 89aa2d7..6ef8932 100644
--- a/gdb/testsuite/ChangeLog
+++ b/gdb/testsuite/ChangeLog
@@ -1,3 +1,7 @@ 
+2016-02-12  Marcin Kościelnicki  <koriakin@0x04.net>
+
+	* gdb.trace/tfile-avx.c (main): Change ymm15 to xmm15.
+
 2016-02-12  Simon Marchi  <simon.marchi@ericsson.com>
 
 	* i386-biarch-core.exp: Define corefile using
diff --git a/gdb/testsuite/gdb.trace/tfile-avx.c b/gdb/testsuite/gdb.trace/tfile-avx.c
index 212c556..3cc3ec0 100644
--- a/gdb/testsuite/gdb.trace/tfile-avx.c
+++ b/gdb/testsuite/gdb.trace/tfile-avx.c
@@ -35,7 +35,9 @@  end (void)
 int
 main (void)
 {
-  register __v8si a asm("ymm15") = {
+  /* Strictly speaking, it should be ymm15 (xmm15 is 128-bit), but gcc older
+     than 4.9 doesn't recognize "ymm15" as a valid register name.  */
+  register __v8si a asm("xmm15") = {
     0x12340001,
     0x12340002,
     0x12340003,