[llvm] r219734 - ARM: remove ARM/Thumb distinction for preferred alignment.

Tim Northover tnorthover at apple.com
Fri Oct 17 14:53:51 PDT 2014


> This commit (or one of the related ones) has introduced a regression for
> Thumb1. The following code should print "0":

Hmm, thanks Oliver. The entire Thumb1 rewriteFrameIndex looks more and more dodgy as I look at it. I'm almost sure I could make it overwrite CPSR too.

I'm also getting more convinced that we do want to try and align local variables to 32-bits if possible, though the code should be able to cope without. Even if DataLayout isn't the way to handle this.

I might get a point fix working today, but I think I need to do some more work here.

Cheers.

Tim.



More information about the llvm-commits mailing list