This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] DATA_ALIGNMENT vs. DATA_ABI_ALIGNMENT (PR target/56564)
- From: DJ Delorie <dj at redhat dot com>
- To: Jakub Jelinek <jakub at redhat dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Mon, 10 Jun 2013 20:44:05 -0400
- Subject: Re: [PATCH] DATA_ALIGNMENT vs. DATA_ABI_ALIGNMENT (PR target/56564)
- References: <20130607192540 dot GH1493 at tucnak dot redhat dot com> <51B245EF dot 3080602 at redhat dot com> <20130607211419 dot GI1493 at tucnak dot redhat dot com> <51B5E80A dot 2040600 at redhat dot com> <20130610154421 dot GU1493 at tucnak dot redhat dot com>
> @@ -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...