On 07/12/2018 01:34 PM, Robin Dapp wrote: > Hi, > >> Please skip '+ && !opts->x_optimize_size)'. I'm attaching patch >> that will >> set opts->x_flag_align_functions to 0 for -Os. It's part of another batch >> alignment patches I'm preparing. > > done in the attached version and added some tests (which do not all fail > without the patch as we can get lucky with the alignment). > > Regtested on s390x. > > Regards > Robin > > -- > > gcc/ChangeLog: > > 2018-07-12 Robin Dapp <rd...@linux.ibm.com> > > * config/s390/s390.c (s390_default_align): Set default function > alignment to 16. > (s390_override_options_after_change): Call s390_default align. > (s390_option_override_internal): Call s390_default_align. > (TARGET_OVERRIDE_OPTIONS_AFTER_CHANGE): Define. > > gcc/testsuite/ChangeLog: > > 2018-07-12 Robin Dapp <rd...@linux.ibm.com> > > * gcc.target/s390/function-align1.c: New test. > * gcc.target/s390/function-align2.c: New test. > * gcc.target/s390/function-align3.c: New test. >
Ok to apply. Thanks! Andreas
