On 06/04/2015 05:05 AM, Renlin Li wrote:
Hi all,

This is a simple patch to change the align variable (used in
emit_local()) from type "int" to type "unsigned int".

It should be defined as "unsigned int" which is the same data type
returned by symtab_node::get (decl)->definition_alignment ()
For the maximum alignment allowed by GCC(without producing "error:
requested alignment is too large"), "int" will overflow.

Okay to Commit on the trunk and backport to branch 5.0?

Regards,
Renlin Li

gcc/ChangeLog:

2015-06-04  Renlin Li  <renlin...@arm.com>

     * varasm.c (emit_local): Use unsigned int for align variable.
OK.
jeff

Reply via email to