> > diff --git a/gcc/config/riscv/riscv.cc b/gcc/config/riscv/riscv.cc > > index 45a63cab9c9..629e5e45cac 100644 > > --- a/gcc/config/riscv/riscv.cc > > +++ b/gcc/config/riscv/riscv.cc > > @@ -5729,7 +5729,8 @@ riscv_get_separate_components (void) > > > > if (riscv_use_save_libcall (&cfun->machine->frame) > > || cfun->machine->interrupt_handler_p > > - || !cfun->machine->frame.gp_sp_offset.is_constant ()) > > + || !cfun->machine->frame.gp_sp_offset.is_constant () > > + || TARGET_ZCMP) > > return components; > > I think this is a bad idea. I have a use case where we use the C > extensions but still compile for -O2 because we want the code to be > fast as possible but still having the savings of the C extensions.
Yeah, agree, so I would prefer to drop this from the patch series. > Thanks, > Andrew Pinski