This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Fix PR 33870
On Nov 14, 2007 1:56 PM, Diego Novillo <dnovillo@google.com> wrote:
> Richard Guenther wrote:
>
> > I guess in add_uids_for_ptset instead of
> >
> > ! if (SFT_NESTING_LEVEL (sft) > 0)
> > ! SFT_UNPARTITIONABLE_P (sft) = true;
> >
> > we can get away with only doing that if the type of the SFT itself could
> > have subvariables (which is probably again what you intended?).
>
> Hmm, well the SFTs themselves are usually of scalar type. The first SFT
> of an int[] array is of type int. Yes, the bug is around here. Perhaps
> the easiest way to fix this is to mark the variable earlier when we are
> pushing fields in the fieldstack (in the code that computes the nesting
> level). And then transport that marker into the SFT here.
Yes, I'm doing that at the moment. Stay tuned for a patch.
Richard.