[AArch64-4.7] Backport: Fix g++.dg/abi/aarch64_guard1.C

[AArch64-4.7] Backport: Fix g++.dg/abi/aarch64_guard1.C

[James Greenhalgh]

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

This is a multi-part message in MIME format.

[-- Attachment #2: Type: text/plain, Size: 602 bytes --]


Hi,

If we enable section anchors by default we must fix the
ABI testcase which is not expecting section anchors.

This was fixed by Andrew Pinski on trunk here:
http://gcc.gnu.org/ml/gcc-patches/2012-11/msg00580.html
So we backport that fix.

I've tested this with no regressions on aarch64-none-elf.

Is this OK to commit to aarch64-4.7-branch?

Thanks,
James Greenhalgh

---
gcc/

2013-02-13  James Greenhalgh  <james.greenhalgh@arm.com>

	Backport from mainline.
	2012-11-06  Andrew Pinski  <apinski@cavium.com>

	* g++.dg/abi/aarch64_guard1.C: Add -fno-section-anchors.

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #3: 0001-AArch64-4.7-Backport-Fix-g-.dg-abi-aarch64_guard1.C.patch --]
[-- Type: text/x-patch;  name=0001-AArch64-4.7-Backport-Fix-g-.dg-abi-aarch64_guard1.C.patch, Size: 535 bytes --]

diff --git a/gcc/testsuite/g++.dg/abi/aarch64_guard1.C b/gcc/testsuite/g++.dg/abi/aarch64_guard1.C
index af82ad2..ca1778b 100644
--- a/gcc/testsuite/g++.dg/abi/aarch64_guard1.C
+++ b/gcc/testsuite/g++.dg/abi/aarch64_guard1.C
@@ -2,7 +2,7 @@
 // 8-byte doubleword and that only the least significant bit is used
 // for initialization guard variables.
 // { dg-do compile { target aarch64*-*-* } }
-// { dg-options "-O -fdump-tree-original" }
+// { dg-options "-O -fdump-tree-original -fno-section-anchors" }
 
 int bar();
 

Re: [AArch64-4.7] Backport: Fix g++.dg/abi/aarch64_guard1.C

[Marcus Shawcroft]

On 13 February 2013 13:28, James Greenhalgh <james.greenhalgh@arm.com> wrote:
>
> Hi,
>
> If we enable section anchors by default we must fix the
> ABI testcase which is not expecting section anchors.
>
> This was fixed by Andrew Pinski on trunk here:
> http://gcc.gnu.org/ml/gcc-patches/2012-11/msg00580.html
> So we backport that fix.
>
> I've tested this with no regressions on aarch64-none-elf.
>
> Is this OK to commit to aarch64-4.7-branch?


Yes

> gcc/

gcc/testsuite


/Marcus