This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH, S390] Increase function alignment to 16 bytes


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  <rdapp@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  <rdapp@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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]