[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