Re: [Patch AArch64] Fix PR target/63874

2016-06-30 Thread Richard Earnshaw (lists)
On 31/03/16 14:11, Ramana Radhakrishnan wrote: > Hi, > > In this PR we have a situation where we aren't really detecting > weak references vs weak definitions. If one has a weak definition > that binds locally there's no reason not to put out PC relative > relocations. > > However if you ha

Re: [Patch AArch64] Fix PR target/63874

2016-06-30 Thread Ramana Radhakrishnan
On Thu, May 5, 2016 at 8:45 AM, Ramana Radhakrishnan wrote: > On Thu, Mar 31, 2016 at 2:11 PM, Ramana Radhakrishnan > wrote: >> Hi, >> >> In this PR we have a situation where we aren't really detecting >> weak references vs weak definitions. If one has a weak definition >> that binds loca

Re: [Patch AArch64] Fix PR target/63874

2016-05-05 Thread Ramana Radhakrishnan
On Thu, Mar 31, 2016 at 2:11 PM, Ramana Radhakrishnan wrote: > Hi, > > In this PR we have a situation where we aren't really detecting > weak references vs weak definitions. If one has a weak definition > that binds locally there's no reason not to put out PC relative > relocations. > > Ho

Re: [Patch AArch64] Fix PR target/63874

2016-04-23 Thread Ramana Radhakrishnan
On Thu, Mar 31, 2016 at 5:30 PM, James Greenhalgh wrote: > On Thu, Mar 31, 2016 at 02:11:49PM +0100, Ramana Radhakrishnan wrote: >> Hi, >> >> In this PR we have a situation where we aren't really detecting >> weak references vs weak definitions. If one has a weak definition >> that binds loc

Re: [Patch AArch64] Fix PR target/63874

2016-03-31 Thread James Greenhalgh
On Thu, Mar 31, 2016 at 02:11:49PM +0100, Ramana Radhakrishnan wrote: > Hi, > > In this PR we have a situation where we aren't really detecting > weak references vs weak definitions. If one has a weak definition > that binds locally there's no reason not to put out PC relative > relocations.

[Patch AArch64] Fix PR target/63874

2016-03-31 Thread Ramana Radhakrishnan
Hi, In this PR we have a situation where we aren't really detecting weak references vs weak definitions. If one has a weak definition that binds locally there's no reason not to put out PC relative relocations. However if you have a genuine weak reference that is known not to bind locally