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

login
register
mail settings
Submitter James Greenhalgh
Date Feb. 13, 2013, 1:28 p.m.
Message ID <1360762115-23188-1-git-send-email-james.greenhalgh@arm.com>
Download mbox | patch
Permalink /patch/220134/
State New
Headers show

Comments

James Greenhalgh - Feb. 13, 2013, 1:28 p.m.
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.
Marcus Shawcroft - Feb. 13, 2013, 1:32 p.m.
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

Patch

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();