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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to