public inbox for gcc-cvs@sourceware.org
help / color / mirror / Atom feed
* [gcc(refs/users/rguenth/heads/vect-force-slp)] Place easily identifyable assert insead of SIGSEV
@ 2024-05-13 14:27 Richard Biener
  0 siblings, 0 replies; 2+ messages in thread
From: Richard Biener @ 2024-05-13 14:27 UTC (permalink / raw)
  To: gcc-cvs

https://gcc.gnu.org/g:a1126dd1b7b0ba051d7d62de2c12b7affa2ecc34

commit a1126dd1b7b0ba051d7d62de2c12b7affa2ecc34
Author: Richard Biener <rguenther@suse.de>
Date:   Fri Mar 1 14:56:01 2024 +0100

    Place easily identifyable assert insead of SIGSEV
    
    Better identification of known ICEs.
    
            * tree-vect-stmts.cc (vect_is_simple_use): Assert instead of
            SIGSEV.

Diff:
---
 gcc/tree-vect-stmts.cc | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/gcc/tree-vect-stmts.cc b/gcc/tree-vect-stmts.cc
index 8fef72cb9072..ca81957def06 100644
--- a/gcc/tree-vect-stmts.cc
+++ b/gcc/tree-vect-stmts.cc
@@ -14009,6 +14009,10 @@ vect_is_simple_use (vec_info *vinfo, stmt_vec_info stmt, slp_tree slp_node,
       *vectype = SLP_TREE_VECTYPE (child);
       if (SLP_TREE_DEF_TYPE (child) == vect_internal_def)
 	{
+	  /* ???  Instead of crashing, easier to identify.  But we
+	     need to think what to do with internal defs of VEC_PERM
+	     kind here.  */
+	  gcc_assert (SLP_TREE_REPRESENTATIVE (child));
 	  *op = gimple_get_lhs (SLP_TREE_REPRESENTATIVE (child)->stmt);
 	  return vect_is_simple_use (*op, vinfo, dt, def_stmt_info_out);
 	}

^ permalink raw reply	[flat|nested] 2+ messages in thread
* [gcc(refs/users/rguenth/heads/vect-force-slp)] Place easily identifyable assert insead of SIGSEV
@ 2024-03-01 15:01 Richard Biener
  0 siblings, 0 replies; 2+ messages in thread
From: Richard Biener @ 2024-03-01 15:01 UTC (permalink / raw)
  To: gcc-cvs

https://gcc.gnu.org/g:8cdfb7016c4ca01c950f358891038f6d57775a02

commit 8cdfb7016c4ca01c950f358891038f6d57775a02
Author: Richard Biener <rguenther@suse.de>
Date:   Fri Mar 1 14:56:01 2024 +0100

    Place easily identifyable assert insead of SIGSEV
    
    Better identification of known ICEs.
    
            * tree-vect-stmts.cc (vect_is_simple_use): Assert instead of
            SIGSEV.

Diff:
---
 gcc/tree-vect-stmts.cc | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/gcc/tree-vect-stmts.cc b/gcc/tree-vect-stmts.cc
index efe40664025..6f834f10c72 100644
--- a/gcc/tree-vect-stmts.cc
+++ b/gcc/tree-vect-stmts.cc
@@ -13972,6 +13972,10 @@ vect_is_simple_use (vec_info *vinfo, stmt_vec_info stmt, slp_tree slp_node,
       *vectype = SLP_TREE_VECTYPE (child);
       if (SLP_TREE_DEF_TYPE (child) == vect_internal_def)
 	{
+	  /* ???  Instead of crashing, easier to identify.  But we
+	     need to think what to do with internal defs of VEC_PERM
+	     kind here.  */
+	  gcc_assert (SLP_TREE_REPRESENTATIVE (child));
 	  *op = gimple_get_lhs (SLP_TREE_REPRESENTATIVE (child)->stmt);
 	  return vect_is_simple_use (*op, vinfo, dt, def_stmt_info_out);
 	}

^ permalink raw reply	[flat|nested] 2+ messages in thread
end of thread, other threads:[~2024-05-13 14:27 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-05-13 14:27 [gcc(refs/users/rguenth/heads/vect-force-slp)] Place easily identifyable assert insead of SIGSEV Richard Biener
  -- strict thread matches above, loose matches on Subject: below --
2024-03-01 15:01 Richard Biener

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).