This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Don't clobber dominator info in the combiner (PR target/77526)
- From: Segher Boessenkool <segher at kernel dot crashing dot org>
- To: Jakub Jelinek <jakub at redhat dot com>
- Cc: Richard Biener <rguenther at suse dot de>, gcc-patches at gcc dot gnu dot org
- Date: Thu, 15 Sep 2016 18:20:11 -0500
- Subject: Re: [PATCH] Don't clobber dominator info in the combiner (PR target/77526)
- Authentication-results: sourceware.org; auth=none
- References: <20160915225044.GC7282@tucnak.redhat.com>
Hi Jakub,
On Fri, Sep 16, 2016 at 12:50:44AM +0200, Jakub Jelinek wrote:
> As mentioned in the PR, combiner sometimes calls
> purge_all_dead_edges or purge_dead_edges that can invalidate the dominator
> info if it is computed. Other passes like CSE in that case free the
> dominance info, this patch does the same in the combiner.
>
> Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?
Okay, but does it need the same for the post-dominators? Oh, those
never exist longer than a single pass?
Thanks,
Segher