]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
lower-bitint: Fix ICE on bitint-39.c
authorJakub Jelinek <jakub@redhat.com>
Fri, 1 Dec 2023 08:26:24 +0000 (09:26 +0100)
committerJakub Jelinek <jakub@redhat.com>
Fri, 1 Dec 2023 08:26:24 +0000 (09:26 +0100)
torture/bitint-39.c ICEs with -O1; the problem is that the
finish_arith_overflow code in one spot replaces use_stmt with an
assignment or cast, but if unlucky and m_gsi iterator is the same statement,
when the code later
      tree clobber = build_clobber (TREE_TYPE (var), CLOBBER_EOL);
      g = gimple_build_assign (var, clobber);
      gsi_insert_after (&m_gsi, g, GSI_SAME_STMT);
it will insert after iterator which contains already replaced statement and
that causes the gimple chain corruption.

2023-12-01  Jakub Jelinek  <jakub@redhat.com>

* gimple-lower-bitint.cc (bitint_large_huge::finish_arith_overflow):
When replacing use_stmt which is gsi_stmt (m_gsi), update m_gsi to
the new statement.

gcc/gimple-lower-bitint.cc

index 09eb7a34e91b6b9a71d0d0f2171708af58fc7efc..70009319fdccc48d54e44b1bb96a9607580fe9df 100644 (file)
@@ -3682,6 +3682,8 @@ bitint_large_huge::finish_arith_overflow (tree var, tree obj, tree type,
              else
                g = gimple_build_assign (lhs2, NOP_EXPR, ovf);
              gsi_replace (&gsi, g, true);
+             if (gsi_stmt (m_gsi) == use_stmt)
+               m_gsi = gsi_for_stmt (g);
              break;
            }
        }