[llvm-commits] [PATCH] Fix ARM var-arg bug (bug 8347)

Evan Cheng evan.cheng at apple.com
Fri Oct 15 10:11:41 PDT 2010


Looks good. Please commit. Thanks.

Evan

On Oct 15, 2010, at 7:20 AM, David Meyer wrote:

> Hi Evan / Jim,
> 
> For reference, X86-64 does the same thing for var-arg (spill registers upon function entry) and the immutable flag is correctly set to false for the register save area.
> 
> This does limit optimization, but unfortunately it is needed for correctness. (at least, without making other changes to the backend)
> 
> - David M
> 
> On Wed, Oct 13, 2010 at 11:59 AM, Jim Grosbach <grosbach at apple.com> wrote:
> Hi David,
> 
> Your explanation definitely makes sense. Thanks for digging into this. My only hesitation is a, perhaps unfounded, concern that this may be a bit of a big hammer and limit other opportunities that are desirable. Evan, what do you think?
> 
> -Jim
> 
> On Oct 13, 2010, at 10:38 AM, David Meyer wrote:
> 
>> Attached patch resolves this bug:
>> 
>> http://llvm.org/bugs/show_bug.cgi?id=8347
>> 
>> - David M
>> <bugfix8347.patch>_______________________________________________
>> 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/20101015/ec9c26eb/attachment.html>


More information about the llvm-commits mailing list