[PATCH] X86_64 target now obeys llc -disable-red-zone option

Steve King steve at metrokings.com
Thu Oct 23 13:12:37 PDT 2014


On Thu, Oct 23, 2014 at 12:00 PM, Jim Grosbach <grosbach at apple.com> wrote:
>
> On Oct 23, 2014, at 11:58 AM, Eric Christopher <echristo at gmail.com> wrote:
>
>
>
> On Thu Oct 23 2014 at 6:22:58 AM Rafael EspĂ­ndola
> <rafael.espindola at gmail.com> wrote:
>>
>> > Interesting. We probably shouldn't have two ways of telling codegen to
>> > not use a redzone. Eric, any thought on it?
>>
>> To be clear, maybe we should just delete DisableRedZone and update the
>> tests that use it to have the attribute?
>
> Seems reasonable to me.  Jim?
>
> SGTM.

To wrap this up:  I will abandon the current patch and submit a new
patch that removes no-red-zone from the LLC command line.  Correct?




More information about the llvm-commits mailing list