On Fri, 11 Nov 2016, Jiong Wang wrote:

>    In this new patch, I introduced a new target macro for SSP to allow one
>   backend GCC's default SSP runtime generation be disabled.

I see no reason this needs to be a target macro rather than a hook.  New 
target macros are discouraged and should only be added for good reasons 
(e.g. used in target-side code; used in defining contents of an enum or 
some other such thing that can't be runtime-conditional; forms part of an 
existing family of target macros).

-- 
Joseph S. Myers
jos...@codesourcery.com

Reply via email to