Committed to trunk as r224333.
Tom: Can you approve merging this to the branch? Llvm-commits seems to be a bit
behind at the moment so here's a link to the Phabricator page
http://reviews.llvm.org/rL224333.
> -----Original Message-----
> From: Sebastian Dreßler [mailto:sebastian.dressler at gmail.com]
> Sent: 16 December 2014 11:07
> To: Daniel Sanders
> Cc: Ben Pope; Nikola Smiljanic; llvmdev
> Subject: Re: [LLVMdev] 3.5.1 Testing Phase Begins
>
> Yeah, I was just puzzled :D No worries.
>
> Cheers,
> Sebastian
>
> 2014-12-16 11:09 GMT+01:00 Daniel Sanders <Daniel.Sanders at
imgtec.com>:
> > It looks like I missed that email completely, I started out replying
to
> > Nikola and Hans which is why I was talking about Windows rather than
OS
> X.
> >
> >
> >
> > I've just found your first email in my llvmdev folder and I think
I see what
> > happened. My mail rules deliver llvmdev messages to a folder named
> llvmdev
> > unless I'm directly addressed in which case it delivers to my
inbox. It
> > looks like you pruned the recipient list when you replied to Tom's
email so
> > that one email was delivered to llvmdev while the rest of the thread
was
> > delivered to my inbox. Then when I replied to Hans and Nikola it seems
I
> > pruned the recipient list (I don't remember doing this and
it's unusual for
> > me but that's what the emails say). You presumably saw my email
and
> thought
> > I was replying to you and things got confused from there.
> >
> >
> >
> > I'll commit that patch to the trunk and ask Tom for permission to
merge it.
> >
> >
> >
> > From: sebastian.dressler at gmail.com
> [mailto:sebastian.dressler at gmail.com]
> > Sent: 16 December 2014 05:20
> > To: Daniel Sanders
> > Cc: Ben Pope; Nikola Smiljanic; llvmdev
> > Subject: Re: [LLVMdev] 3.5.1 Testing Phase Begins
> >
> >
> >
> >
> >
> > On 15.12.2014, at 23:01, Daniel Sanders <Daniel.Sanders at
imgtec.com>
> wrote:
> >
> >
> >
> > From: sebastian.dressler at gmail.com [sebastian.dressler at
gmail.com]
> > Sent: 15 December 2014 18:41
> > To: Daniel Sanders
> > Cc: Ben Pope; Nikola Smiljanic; llvmdev
> > Subject: Re: [LLVMdev] 3.5.1 Testing Phase Begins
> >
> > For sake of completeness, I copy the output again:
> >
> > Command Output (stderr): […]
> >
> >
> > That's given me enough to figure out the problem. %gp_rel is only
used
> when
> > the small data section is enabled and that's only supposed to be
the case
> > for bare-metal. However, the guard is actually a test for 'is not
Linux' and
> > this condition is of course true for windows.
> >
> >
> >
> > Just out of curiosity: this information was also contained in my
initial
> > mail, did you probably overlook it?
> >
> >
> >
> > It's not a new bug (it was in 3.5.0 too) but the testcase is new.
Does the
> > attached patch fix the problem for you?<arguments-struct.patch>
> >
> >
> >
> > Yes, it fixes it.
> >
> >
> >
> >
> >
> > Cheers,
> >
> > Sebastian
> >
> >
> >
> >