On Sun, 2021-04-25 at 20:50 -0500, Bill Schmidt via Gcc-patches wrote: > 2021-03-25 Bill Schmidt <wschm...@linux.ibm.com> > > gcc/ > * config/rs6000/rs6000-c.c (rs6000_target_modify_macros): Define > __ROP_PROTECT__ if -mrop-protect is selected.
ok > --- > gcc/config/rs6000/rs6000-c.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/gcc/config/rs6000/rs6000-c.c b/gcc/config/rs6000/rs6000-c.c > index 0f8a629ff5a..afcb5bb6e39 100644 > --- a/gcc/config/rs6000/rs6000-c.c > +++ b/gcc/config/rs6000/rs6000-c.c > @@ -602,6 +602,9 @@ rs6000_target_modify_macros (bool define_p, HOST_WIDE_INT > flags, > /* Whether pc-relative code is being generated. */ > if ((flags & OPTION_MASK_PCREL) != 0) > rs6000_define_or_undefine_macro (define_p, "__PCREL__"); > + /* Tell the user -mrop-protect is in play. */ > + if (rs6000_rop_protect) > + rs6000_define_or_undefine_macro (define_p, "__ROP_PROTECT__"); > I notice that almost all of the other defines are controled by an (if (flags & OPTION) logic block.. but this seems OK. lgtm, thanks, -WIll > } > > void
