[PATCH]Use "unsigned int" instead of "int" to hold alignment in emit_local function.

Renlin Li renlin.li@arm.com
Thu Jun 4 11:28:00 GMT 2015


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.li@arm.com>

     * varasm.c (emit_local): Use unsigned int for align variable.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: new.diff
Type: text/x-patch
Size: 774 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20150604/cf19ce6b/attachment.bin>


More information about the Gcc-patches mailing list