[llvm] r185393 - Remove floating point computations form SpillPlacement.cpp.

Jakob Stoklund Olesen stoklund at 2pi.dk
Tue Jul 2 10:35:18 PDT 2013


On Jul 2, 2013, at 5:48 AM, Renato Golin <renato.golin at linaro.org> wrote:

> Hi Jakob, Benjamin,
> 
> Just FYI, this commit broke Lencod on ARM again, though, if this is the same problem as before, it seems that Manman and Stepan already have a solution in mind.
> 
> I'm not sure the current problem is, again, due to spilling the wrong offsets on the ByVal stack, but just to let you know, in case it doesn't, we might have to investigate this commit further as well. ;)

Thanks, Renato.

I’ve reverted the patch in r185447 while we investigate. I talked to Manman about the byval problem, and it does sound like a likely culprit. I’ll reapply the patch once the byval problem has been fixed.

Thanks,
/jakob





More information about the llvm-commits mailing list