> 2018-03-22 Renlin Li <renlin...@arm.com> > > PR middle-end/84877 > * explow.h (get_dynamic_stack_size): Declare it as external. > * explow.c (record_new_stack_level): Remove function static attribute. > * function.c (assign_stack_local_1): Dynamically align the stack slot > addr for parameter copy on the stack. > > gcc/testsuite/ > > 2018-03-22 Renlin Li <renlin...@arm.com> > > PR middle-end/84877 > * gcc.dg/pr84877.c: New.
As reported by Rainer, the test doesn't pass on several targets. And the patch breaks 64-bit Windows, see PR bootstrap/88450. More generally, the patch totally disregards the SUPPORTS_STACK_ALIGNMENT machinery of the middle-end, which is quite surprising given that this machinery was precisely designed for this purpose. Why is it so? -- Eric Botcazou