[PATCH] tree-optimization/103188 - avoid running ranger on not-up-to-date SSA

Aldy Hernandez aldyh@redhat.com
Thu Nov 11 17:09:01 GMT 2021


Like this.  It simplifies both loop-ch and the threader.

I'll push this pending tests unless someone objects.

On Thu, Nov 11, 2021 at 5:43 PM Aldy Hernandez <aldyh@redhat.com> wrote:
>
> Thanks for doing this!
>
> >
> > +  gimple_ranger *ranger = new gimple_ranger;
> > +  path_range_query *query = new path_range_query (*ranger, /*resolve=*/true);
>
> Hmmm, it looks like both clients are now instantiating a gimple_ranger
> just so they can pass it down to the path_range_query.  Maybe we
> should  have another ctor with just:
>
> path_range_query (bool resolve);
>
> ...and have it allocate its own ranger.
>
> Does this seem like a useful improvement?  For that matter, resolve
> should default to true.  The option is only there so the backward
> threader can run in a "light" mode (early threading, etc).
>
> Aldy
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Make-ranger-optional-in-path_range_query.patch
Type: text/x-patch
Size: 8552 bytes
Desc: not available
URL: <https://gcc.gnu.org/pipermail/gcc-patches/attachments/20211111/1e8c07de/attachment-0001.bin>


More information about the Gcc-patches mailing list