llvm dev - Dec 2014 - [LLVMdev] 3.5.1 Testing Phase Begins

> On 15.12.2014, at 15:45, Daniel Sanders <Daniel.Sanders at
imgtec.com> wrote:
> 
>> -----Original Message-----
>> From: llvmdev-bounces at cs.uiuc.edu [mailto:llvmdev-bounces at
cs.uiuc.edu]
>> On Behalf Of Sebastian Dreßler
>> Sent: 15 December 2014 14:08
>> Cc: Ben Pope; Nikola Smiljanic; llvmdev
>> Subject: Re: [LLVMdev] 3.5.1 Testing Phase Begins
>> 
>> 2014-12-15 13:00 GMT+01:00 Daniel Sanders <Daniel.Sanders at
imgtec.com>:
>>>> LLVM: CodeGen/Mips/cconv/arguments-struct.ll
>>> 
>>> Do you have the verbose lit output for this test?
>>> 
>> 
>> I ran it again in verbose mode, but there were no additional outputs.
>> 
>> Cheers,
>> Sebastian
> 
> Just to double check: Is that with -v or -vv? The latter should print the
output of the failing command.
It was -v but I ran again with -vv. However, there is no new information. 

For sake of completeness, I copy the output again:




Command Output (stderr):
--
/Users/rock/llvm/3.5.1-testing/rc1/llvm.src/test/CodeGen/Mips/cconv/arguments-struct.ll:28:14:
error: expected string not found in input
; SYM32-DAG: lui [[PTR_HI:\$[0-9]+]], %hi(bytes)
             ^
<stdin>:15:7: note: scanning from here
s_i8: # @s_i8
      ^
<stdin>:24:5: note: possible intended match here
 addiu $2, $gp, %gp_rel(bytes)
    ^

—

Cheers,
Sebastian
> 
>> _______________________________________________
>> LLVM Developers mailing list
>> LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev

> 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
> 
> > On 15.12.2014, at 15:45, Daniel Sanders <Daniel.Sanders at
imgtec.com> wrote:
> >
> >> -----Original Message-----
> >> From: llvmdev-bounces at cs.uiuc.edu [mailto:llvmdev-bounces at
cs.uiuc.edu]
> >> On Behalf Of Sebastian Dreßler
> >> Sent: 15 December 2014 14:08
> >> Cc: Ben Pope; Nikola Smiljanic; llvmdev
> >> Subject: Re: [LLVMdev] 3.5.1 Testing Phase Begins
> >>
> >> 2014-12-15 13:00 GMT+01:00 Daniel Sanders <Daniel.Sanders at
imgtec.com>:
> >>>> LLVM: CodeGen/Mips/cconv/arguments-struct.ll
> >>>
> >>> Do you have the verbose lit output for this test?
> >>>
> >>
> >> I ran it again in verbose mode, but there were no additional
outputs.
> >>
> >> Cheers,
> >> Sebastian
> >
> > Just to double check: Is that with -v or -vv? The latter should print
the output of the failing command.
> 
> It was -v but I ran again with -vv. However, there is no new information.
> 
> For sake of completeness, I copy the output again:
> 
> Command Output (stderr):
> --
>
/Users/rock/llvm/3.5.1-testing/rc1/llvm.src/test/CodeGen/Mips/cconv/arguments-struct.ll:28:14:
error: expected string not found in input
> ; SYM32-DAG: lui [[PTR_HI:\$[0-9]+]], %hi(bytes)
>              ^
> <stdin>:15:7: note: scanning from here
> s_i8: # @s_i8
>       ^
> <stdin>:24:5: note: possible intended match here
>  addiu $2, $gp, %gp_rel(bytes)
>     ^
> 
> —
> 
> Cheers,
> Sebastian
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.

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?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: arguments-struct.patch
Type: text/x-patch
Size: 2853 bytes
Desc: arguments-struct.patch
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20141215/fe8fbfae/attachment.bin>

> 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


-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20141216/2ffb6b6b/attachment.html>