https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52272

amker at gcc dot gnu.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |amker at gcc dot gnu.org

--- Comment #27 from amker at gcc dot gnu.org ---
I proposed a new approach fixing PR50955.  In that way, we won't have this
regression anymore.
In this comment I want to compare that proposal to the one given by Richard in
comment #5.

Conclusion is, I think that proposal doesn't bring any more obvious benefit
comparing to this one.  Yet this one can handle case with many address type
iv_uses which share common sub-expression except for base_object.

Of course this patch has its own disadvantages.  For example, it needs to add
more candidates.  And since we don't do analysis when adding candidate (that's
in candidate selection), we actually need to add candidates that might be
useful.

We need to analysis iv_uses to add candidates only when necessary as much as
possible.  We also need to handle complexity issue caused by more candidates.

Reply via email to