This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [graphite] Fix ICE in aermod.f90
- From: Richard Guenther <rguenther at suse dot de>
- To: Sebastian Pop <sebpop at gmail dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, "Jagasia, Harsha" <harsha dot jagasia at amd dot com>
- Date: Wed, 21 Jan 2009 10:09:54 +0100 (CET)
- Subject: Re: [graphite] Fix ICE in aermod.f90
- References: <cb9d34b20901201130v7afad197tc02fe5a21d6807de@mail.gmail.com>
On Tue, 20 Jan 2009, Sebastian Pop wrote:
> Hi,
>
> The problem here is latent in trunk, but exposed in graphite branch by
> the fact that we have LIM scheduled just after Graphite. The cleanup
> of the CFG does not maintain the loop closed SSA property, and thus we
> have to recompute it after the CFG cleanup. Regstrap in progress on
> amd64-linux. Okay for trunk once this finishes?
Are you sure? It works for complete loop unrolling - maybe you forgot
to update virtual SSA form?
Btw, I recently noticed that you call update-ssa _way_ too often
and recompute dominators _way_ too often in graphite.c
Richard.