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

Davide Italiano via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 24 16:28:17 PDT 2017


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?

-- 
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