Commit: Fix display of 64-bit CFI and FDE frames

[Kyrill Tkachov]

Hi Nick, all,

Seems that an aarch64 test also needs updating to match the 64-bit 
addresses.
Otherwise we get a FAIL on aarch64.
This patch updates the test.

ran check-ld on aarch64-none-elf to confirm that it works.

Ok for trunk?

Thanks,
Kyrill

[ld/testsuite/]

2013-09-16  Kyrylo Tkachov  <kyrylo.tkachov@arm.com>

     * ld-aarch64/eh-frame.d: Update expected output to allow for
     64-bit addresses.

On 12/09/13 10:13, Nick Clifton wrote:
> Hi Guys,
>
>    Following on from my recent patch to fix the detection of 64-bit CFI
>    and FDE frames, I am now checking in the attached patch to fix the
>    display of 64-bit CFI and FDE information.  The patch also involves
>    updating the testsuites to handle the 64-bit addresses when they are
>    displayed.  I have also added a test to the SAFE_BYTE_GET macro to
>    make sure that the variable receiving the bytes is wide enough to
>    contain all of the bytes that are going to be read.
>
> Cheers
>    Nick
>
> binutils/ChangeLog
> 2013-09-12  Nick Clifton  <nickc@redhat.com>
>
> 	* dwarf.c (dwarf_vmatoa): Rename to dwarf_vmatoa_1 and add a
> 	precision parameter.
> 	(dwarf_vmatoa): New wrapper for dwarf_vmatoa_1.
> 	(print_dwarf_vma): Use dwarf_vmatoa_1.
> 	(SAFE_BYTE_GET): Add check that VAL is big enough to contain
> 	AMOUNT bytes.
> 	(process_debug_info): Use an unsigned int for the offset size.
> 	(process_debug_pubnames): Likewise.
> 	(display_debug_aranges): Likewise.
> 	(struct Frame_Chunk): Use dwarf_vma type for pc_begin and pc_range
> 	fields.
> 	(frame_display_row): Use print_dwarf_vma to display dwarf_vma
> 	values.
> 	(display_debug_frames): Likewise.
>
> binutils/testsuite/ChangeLog
> 2013-09-12  Nick Clifton  <nickc@redhat.com>
>
> 	* binutils-all/x86-64/compressed-1a.d: Update expected output to
> 	allow for 64-bit addresses.
>
> gas/testsuite/ChangeLog
> 2013-09-12  Nick Clifton  <nickc@redhat.com>
>
> 	* gas/cfi/cfi-alpha-1.d: Update expected output to allow for
> 	64-bit addresses.
> 	* gas/cfi/cfi-alpha-3.d: Likewise.
> 	* gas/cfi/cfi-arm-1.d: Likewise.
> 	* gas/cfi/cfi-common-1.d: Likewise.
> 	* gas/cfi/cfi-common-2.d: Likewise.
> 	* gas/cfi/cfi-common-3.d: Likewise.
> 	* gas/cfi/cfi-common-4.d: Likewise.
> 	* gas/cfi/cfi-common-5.d: Likewise.
> 	* gas/cfi/cfi-common-6.d: Likewise.
> 	* gas/cfi/cfi-common-7.d: Likewise.
> 	* gas/cfi/cfi-hppa-1.d: Likewise.
> 	* gas/cfi/cfi-i386-2.d: Likewise.
> 	* gas/cfi/cfi-i386.d: Likewise.
> 	* gas/cfi/cfi-m68k.d: Likewise.
> 	* gas/cfi/cfi-mips-1.d: Likewise.
> 	* gas/cfi/cfi-ppc-1.d: Likewise.
> 	* gas/cfi/cfi-s390-1.d: Likewise.
> 	* gas/cfi/cfi-s390x-1.d: Likewise.
> 	* gas/cfi/cfi-sh-1.d: Likewise.
> 	* gas/cfi/cfi-sparc-1.d: Likewise.
> 	* gas/cfi/cfi-sparc64-1.d: Likewise.
> 	* gas/cfi/cfi-x86_64.d: Likewise.
>
> ld/testsuite/ChangeLog
> 2013-09-12  Nick Clifton  <nickc@redhat.com>
>
> 	* ld-elf/eh1.d: Update expected output to allow for
> 	64-bit addresses.
> 	* ld-elf/eh2.d: Likewise.
> 	* ld-elf/eh3.d: Likewise.
> 	* ld-elf/eh4.d: Likewise.
> 	* ld-elf/eh5.d: Likewise.
> 	* ld-elf/eh6.d: Likewise.
> 	* ld-mips-elf/eh-frame1-n64.d: Likewise.
> 	* ld-mips-elf/eh-frame2-n64.d: Likewise.
> 	* ld-mips-elf/eh-frame3.d: Likewise.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: aarch64-ld-test.patch
Type: text/x-patch
Size: 1137 bytes
Desc: not available
URL: <https://sourceware.org/pipermail/binutils/attachments/20130916/07e6ccc8/attachment.bin>