[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