[llvm] r183552 - Add a backend option to warn on a given stack size limit.

Quentin Colombet qcolombet at apple.com
Fri Jun 7 17:12:45 PDT 2013


I have reapplied the patch (r183595) with what I suppose should fix  the problem on Linux (I reproduce a similar behavior but I do not have a Linux system to test it).

I will keep an eye on the buildbot and if this does not fix the issue, I will revert the patch.

Thanks,

-Quentin

PS: Actually both ARM and X86 were failing, the files that were failing have the same name, so one is overriding the other.

On Jun 7, 2013, at 3:16 PM, Quentin Colombet <qcolombet at apple.com> wrote:

> Reverted in r183579.
> 
> -Quentin
> 
> On Jun 7, 2013, at 3:09 PM, Rafael EspĂ­ndola <rafael.espindola at gmail.com> wrote:
> 
>> On 7 June 2013 18:02, Quentin Colombet <qcolombet at apple.com> wrote:
>>> Yes, I am looking into it.
>>> 
>>> The strange thing here is that it looks like the option does not parse
>>> correctly for ARM:
>>> 
>>> llc: for the -warn-stack-size option: Cannot find option named '80'!
>>> 
>>> Whereas it works perfectly for X86.
>>> 
>>> Any ideas?
>> 
>> Unfortunately no. Would you mind reverting it for now?
>> 
>>> -Quentin
>> 
>> Thanks,
>> Rafael
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130607/1e87e4f0/attachment.html>


More information about the llvm-commits mailing list