On Mon, Jun 10, 2013 at 08:44:05PM -0400, DJ Delorie wrote: > > > @@ -986,12 +1053,10 @@ align_variable (tree decl, bool dont_out > > if (! DECL_THREAD_LOCAL_P (decl) || const_align <= BITS_PER_WORD) > > align = const_align; > > } > > -#endif > > } > > +#endif > > I think this change in get_variable_align() is wrong; it results in > unbalanced braces inside an #ifdef, if the #ifdef body is not included > (i.e. CONSTANT_ALIGNMENT not defined), the compile fails...
You are right, fixed thusly, committed as obvious: 2013-06-11 Jakub Jelinek <ja...@redhat.com> PR target/56564 * varasm.c (get_variable_align): Move #endif to the right place. --- gcc/varasm.c (revision 199933) +++ gcc/varasm.c (revision 199934) @@ -1053,8 +1053,8 @@ get_variable_align (tree decl) if (! DECL_THREAD_LOCAL_P (decl) || const_align <= BITS_PER_WORD) align = const_align; } - } #endif + } return align; } Jakub