[llvm] r298734 - [Outliner] Remove no red zone requirment for AArch64

Davide Italiano via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 29 13:48:50 PDT 2017


On Fri, Mar 24, 2017 at 4:28 PM, Davide Italiano <davide at freebsd.org> wrote:
> On Fri, Mar 24, 2017 at 1:47 PM, Jessica Paquette via llvm-commits
> <llvm-commits at lists.llvm.org> wrote:
>> Author: paquette
>> Date: Fri Mar 24 15:47:59 2017
>> New Revision: 298734
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=298734&view=rev
>> Log:
>> [Outliner] Remove no red zone requirment for AArch64
>>
>> AArch64 doesn't require -mno-red-zone; stack fixups are sufficient here. This was
>> unnecessarily copied over from the X86 target.
>>
>> (You can now outline with red zones! Yay!)
>>
>> Removing the requirement passes all Single/MultiSource tests.
>>
>>
>
> Commenting on this one, even though you reverted. This is clearly a
> functional change and show be accompanied by a test, same for the X86
> bits. Can you please add one?
>

ping?


-- 
Davide

"There are no solved problems; there are only problems that are more
or less solved" -- Henri Poincare


More information about the llvm-commits mailing list