On 01/27/2017 01:02 PM, Eric Botcazou wrote:
The attached patch is a middle ground between the previously working and currently broken situations: if the back-end defines STACK_DYNAMIC_OFFSET, then the middle-end assumes that STACK_DYNAMIC_OFFSET maintains the alignment; if it doesn't, which means the default value of STACK_DYNAMIC_OFFSET computed in function.c is used instead, then the middle-end maintains the alignment itself.
I'd say let's not have a middle ground - this stuff is sufficiently brain-twisting that I'd rather go back to a known working state. If there was an error in the previous patch, let's roll it back until we fully understand the situation.
Bernd