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

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

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

[PATCH, S390] Increase function alignment to 16 bytes

2018-07-11 Thread Robin Dapp
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. Apparently ia64 did it to set flags that are reset when using