public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* [patch gas]: Testsuite patch for cfi
@ 2011-02-09 18:42 Kai Tietz
  2011-02-10 15:22 ` Nick Clifton
  0 siblings, 1 reply; 3+ messages in thread
From: Kai Tietz @ 2011-02-09 18:42 UTC (permalink / raw)
  To: Binutils; +Cc: Nick Clifton

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

Hello,

as x64 PE+ uses different return address column value (actual 32 due
different callee saved register), the test about cfi for x86_64 needs
to be adjusted.

2011-02-09  Kai Tietz

	* testsuite/gas/cfi/cfi-x86_64.d: Adjust for x64 PE+.

Tested for x86_64-pc-linux-gnu and x86_64-w64-mingw32. Ok for apply?

Regards,
Kai

[-- Attachment #2: tcfi.txt --]
[-- Type: text/plain, Size: 1178 bytes --]

Index: testsuite/gas/cfi/cfi-x86_64.d
===================================================================
RCS file: /cvs/src/src/gas/testsuite/gas/cfi/cfi-x86_64.d,v
retrieving revision 1.14
diff -u -3 -r1.14 cfi-x86_64.d
--- testsuite/gas/cfi/cfi-x86_64.d	2 Sep 2010 22:43:46 -0000	1.14
+++ testsuite/gas/cfi/cfi-x86_64.d	9 Feb 2011 18:37:03 -0000
@@ -8,11 +8,11 @@
   Augmentation:          "zR"
   Code alignment factor: 1
   Data alignment factor: -8
-  Return address column: 16
+  Return address column: (16|32)
   Augmentation data:     1b
 
   DW_CFA_def_cfa: r7 \(rsp\) ofs 8
-  DW_CFA_offset: r16 \(rip\) at cfa-8
+  DW_CFA_offset: (r16 \(rip\)|r32 \(xmm15\)) at cfa-8
   DW_CFA_nop
   DW_CFA_nop
 
@@ -56,7 +56,7 @@
   Augmentation:          "zR"
   Code alignment factor: 1
   Data alignment factor: -8
-  Return address column: 16
+  Return address column: (16|32)
   Augmentation data:     1b
 
   DW_CFA_def_cfa: r7 \(rsp\) ofs 8
@@ -89,7 +89,7 @@
   Augmentation:          "zR"
   Code alignment factor: 1
   Data alignment factor: -8
-  Return address column: 16
+  Return address column: (16|32)
   Augmentation data:     1b
 
   DW_CFA_undefined: r16 \(rip\)

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

* Re: [patch gas]: Testsuite patch for cfi
  2011-02-09 18:42 [patch gas]: Testsuite patch for cfi Kai Tietz
@ 2011-02-10 15:22 ` Nick Clifton
  2011-02-10 17:07   ` Kai Tietz
  0 siblings, 1 reply; 3+ messages in thread
From: Nick Clifton @ 2011-02-10 15:22 UTC (permalink / raw)
  To: Kai Tietz; +Cc: Binutils

Hi Kai,

> 2011-02-09  Kai Tietz
>
> 	* testsuite/gas/cfi/cfi-x86_64.d: Adjust for x64 PE+.

Approved - please apply.

Cheers
   Nick


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

* Re: [patch gas]: Testsuite patch for cfi
  2011-02-10 15:22 ` Nick Clifton
@ 2011-02-10 17:07   ` Kai Tietz
  0 siblings, 0 replies; 3+ messages in thread
From: Kai Tietz @ 2011-02-10 17:07 UTC (permalink / raw)
  To: Nick Clifton; +Cc: Binutils

2011/2/10 Nick Clifton <nickc@redhat.com>:
> Hi Kai,
>
>> 2011-02-09  Kai Tietz
>>
>>        * testsuite/gas/cfi/cfi-x86_64.d: Adjust for x64 PE+.
>
> Approved - please apply.
>
> Cheers
>  Nick

Applied.

Thanks,
Kai

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

end of thread, other threads:[~2011-02-10 17:07 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-02-09 18:42 [patch gas]: Testsuite patch for cfi Kai Tietz
2011-02-10 15:22 ` Nick Clifton
2011-02-10 17:07   ` Kai Tietz

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).