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

2018-07-13 Thread Andreas Krebbel
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 t

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

2018-07-12 Thread Robin Dapp
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 ca

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

2018-07-12 Thread Martin Liška
On 07/11/2018 05:40 PM, Robin Dapp wrote: Hi, the following patch increases the default function alignment to 16 bytes. This helps get rid of some unwanted performance effects. I'm unsure whether or when it's necessary to implement OVERRIDE_OPTIONS_AFTER_CHANGE. Hi. Yes, it's how that shoul