On 10/19/21 12:55 PM, Richard Biener wrote:
On Tue, Oct 19, 2021 at 10:48 AM Aldy Hernandez <al...@redhat.com> wrote:

As suggested.  Thanks.

gcc/ChangeLog:

         * passes.def: Change threading comment before pass_ccp pass.
---
  gcc/passes.def | 4 +---
  1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/gcc/passes.def b/gcc/passes.def
index c11c237f6d2..4c54176328b 100644
--- a/gcc/passes.def
+++ b/gcc/passes.def
@@ -339,9 +339,7 @@ along with GCC; see the file COPYING3.  If not see
        NEXT_PASS (pass_thread_jumps);
        NEXT_PASS (pass_vrp, false /* warn_array_bounds_p */);
        NEXT_PASS (pass_vrp_threader);
-      /* Threading can leave many const/copy propagations in the IL.
-        Clean them up.  Instead of just copy_prop, we use ccp to
-        compute alignment and nonzero bits.  */

Well, the comment is still true as long as pass_vrp_threader is there ;)

Ha ha!  Oops.  You're right.

Tell you what, if we decide not to rearrange the pipeline, I'll put the comment back :).

Aldy

Reply via email to