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

Rafael EspĂ­ndola rafael.espindola at gmail.com
Thu Oct 23 06:15:48 PDT 2014


On 22 October 2014 20:57, Steve King <steve at metrokings.com> wrote:
> Rafael - Most of the Target options have the name in the comment.  Are
> you sure you want me to remove?

Yes please. Repeating the name is the old style, and it easily gets out of sync.

> Jim - Clang looks fine.  Setting -mno-red-zone on clang's command line
> sets corresponding function attribute.  The function attributes were
> already visible to the target and at least x86_64 already checked for
> it.

Interesting. We probably shouldn't have two ways of telling codegen to
not use a redzone. Eric, any thought on it?

Cheers,
Rafael



More information about the llvm-commits mailing list