public inbox for gcc-bugs@sourceware.org help / color / mirror / Atom feed
From: "rguenth at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org> To: gcc-bugs@gcc.gnu.org Subject: [Bug c++/106841] [12/13 Regression] ICE in vect_get_vec_defs_for_operand, at tree-vect-stmts.cc:1509 since r12-2733-g31855ba6b16cd138 Date: Tue, 06 Sep 2022 08:01:32 +0000 [thread overview] Message-ID: <bug-106841-4-8CTeuQmNRp@http.gcc.gnu.org/bugzilla/> (raw) In-Reply-To: <bug-106841-4@http.gcc.gnu.org/bugzilla/> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106841 Richard Biener <rguenth at gcc dot gnu.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|unassigned at gcc dot gnu.org |rguenth at gcc dot gnu.org Status|NEW |ASSIGNED --- Comment #4 from Richard Biener <rguenth at gcc dot gnu.org> --- The issue seems to be that we get t.ii:40:17: note: vect_recog_widen_mult_pattern: detected: _56 = _55 * 8; t.ii:40:17: note: widen_mult pattern recognized: patt_36 = (long unsigned int) patt_38; t.ii:40:17: note: extra pattern stmt: patt_38 = _76 w* 8; ... t.ii:40:17: note: ------>vectorizing statement: _55 = (long unsigned int) _76; t.ii:40:17: note: ------>vectorizing statement: patt_63 = _76 w* 8; t.ii:40:17: note: transform statement. ... t.ii:40:17: note: ------>vectorizing statement: A$z_42 = MEM[(const struct R3 &)_57].z; t.ii:40:17: note: multiple-types. t.ii:40:17: note: transform statement. t.ii:40:17: note: vect_is_simple_use: operand (long unsigned int) _76, type of def: internal t.ii:40:17: note: vect_is_simple_use: vectype vector(2) long unsigned int t.ii:40:17: note: transform load. ncopies = 2 t.ii:40:17: note: vect_get_vec_defs_for_operand: _55 and a SLP tree that seems to run into a bug with hybrid discovery: t.ii:40:17: note: node 0x654a7d8 (max_nunits=4, refcnt=2) vector(4) int t.ii:40:17: note: op template: _76 = (int) _75; t.ii:40:17: note: stmt 0 _76 = (int) _75; t.ii:40:17: note: stmt 1 _84 = (int) _83; t.ii:40:17: note: children 0x654a860 t.ii:40:17: note: Processing hybrid candidate : _55 = (long unsigned int) _76; t.ii:40:17: note: Marked SLP consumed stmt pure: _55 = (long unsigned int) _76; that is because the use of _55 is _56 = _55 * 8; but that is part of a pattern that is PURE_SLP and we fail to see the direct use in the gather since gather discovery skips this scaling stmt. We can probably detect this after the fact but the best fix would be to somehow mark _55 as hybrid, I'll see if I can manage to do that.
next prev parent reply other threads:[~2022-09-06 8:01 UTC|newest] Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-09-05 19:14 [Bug c++/106841] New: ice in vect_get_vec_defs_for_operand, at tree-vect-stmts.cc:1509 dcb314 at hotmail dot com 2022-09-05 19:17 ` [Bug c++/106841] " dcb314 at hotmail dot com 2022-09-05 21:17 ` dcb314 at hotmail dot com 2022-09-06 4:49 ` [Bug c++/106841] [12/13 Regression] ICE in vect_get_vec_defs_for_operand, at tree-vect-stmts.cc:1509 since r12-2733-g31855ba6b16cd138 marxin at gcc dot gnu.org 2022-09-06 8:01 ` rguenth at gcc dot gnu.org [this message] 2022-09-06 8:55 ` cvs-commit at gcc dot gnu.org 2022-09-06 9:10 ` [Bug c++/106841] [12 " rguenth at gcc dot gnu.org 2022-09-09 9:50 ` cvs-commit at gcc dot gnu.org 2022-09-09 9:50 ` rguenth at gcc dot gnu.org 2022-10-06 15:56 ` kenneth.hoste at ugent dot be 2022-10-07 6:55 ` rguenth at gcc dot gnu.org
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=bug-106841-4-8CTeuQmNRp@http.gcc.gnu.org/bugzilla/ \ --to=gcc-bugzilla@gcc.gnu.org \ --cc=gcc-bugs@gcc.gnu.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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).