Patchwork Refactor type handling in get_alias_set, fix PR58513

login
register
mail settings
Submitter James Greenhalgh
Date Sept. 26, 2013, 2:13 p.m.
Message ID <1380204780-9503-1-git-send-email-james.greenhalgh@arm.com>
Download mbox | patch
Permalink /patch/278206/
State New
Headers show

Comments

James Greenhalgh - Sept. 26, 2013, 2:13 p.m.
On Tue, Sep 24, 2013 at 12:00:48PM +0100, Richard Biener wrote:
> 2013-09-24  Richard Biener  <rguenther@suse.de>
>
>         * g++.dg/vect/pr58513.cc: New testcase.
>

Hi,

This testcase fails for arm and aarch64 targets when using -fPIC.
As discussed on IRC this can be fixed by making op static.

After asking Richard Earnshaw to explain -fPIC and dynamic linking to
me, I've committed this (now obvious) patch as r202947.

Thanks,
James

---
gcc/testsuite/

2013-09-26  James Greenhalgh  <james.greenhalgh@arm.com>

	* g++.dg/vect/pr58513.cc (op): Make static.

Patch

diff --git a/gcc/testsuite/g++.dg/vect/pr58513.cc b/gcc/testsuite/g++.dg/vect/pr58513.cc
index 2563047..08a175c 100644
--- a/gcc/testsuite/g++.dg/vect/pr58513.cc
+++ b/gcc/testsuite/g++.dg/vect/pr58513.cc
@@ -1,7 +1,7 @@ 
 // { dg-do compile }
 // { dg-require-effective-target vect_int }
 
-int op (const int& x, const int& y) { return x + y; }
+static int op (const int& x, const int& y) { return x + y; }
 
 void foo(int* a)
 {