[PATCH ARM]: PR67745: Fix function alignment after __attribute__ 2/2

Jeff Law law@redhat.com
Wed Sep 30 19:30:00 GMT 2015


On 09/29/2015 07:24 AM, Christian Bruel wrote:
> This patch uses FUNCTION_BOUNDARY instead of DECL_ALIGN to check the max
> align when optimizing for size in assemble_start_function.
> This is necessary for ARM that can switch the max code alignment
> directives between modes.
>
> No regressions for ARM
> Testing on-going for x86
>
> Christian
>
>
> align2.patch
>
>
> 2015-09-29  Christian Bruel<christian.bruel@st.com>
>
> 	PR target/67745
> 	* gcc/varasm.c (assemble_start_function): Use current's function align.
Does this override alignment information that might be attached to the 
DECL?   Does that, in effect, override any alignment information that 
the developer may have put on the decl?  If so, then it seems like a bad 
idea, even with -Os.

Am I missing something here?

jeff



More information about the Gcc-patches mailing list