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

Rafael EspĂ­ndola rafael.espindola at gmail.com
Fri Oct 24 05:04:21 PDT 2014


On 23 October 2014 16:12, Steve King <steve at metrokings.com> wrote:
> 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?

Yes, thanks!




More information about the llvm-commits mailing list