[PATCH] D11815: Pass subtarget feature "force-align-stack"

hfinkel@anl.gov via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 13 17:06:45 PDT 2015


hfinkel added a comment.

In http://reviews.llvm.org/D11815#224161, @echristo wrote:

> Hi Hal,
>
> No, TargetOptions is exactly the wrong place to handle this due to wanting to have various functions compiled with and without a force aligned stack at the IR level that might not hold up at LTO time.


No, that's not a problem. The code in lib/Target/TargetMachine.cpp (specifically via the RESET_OPTION macro), resets the target options based on the function attributes.

> -eric



http://reviews.llvm.org/D11815





More information about the cfe-commits mailing list