On Mon, Apr 26, 2021 at 11:03:22AM -0500, will schmidt wrote: > On Sun, 2021-04-25 at 20:50 -0500, Bill Schmidt via Gcc-patches wrote: > > @@ -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.
That is rs6000_isa_flags, which is only 64 bits. Not every option can be put in there, only the ones used all over the place in the compiler should be. Currently there are 53 of 64 bits used, there is some room, and that is good :-) Segher
