[Bug target/109733] [14 Regression] ICE in extract_insn, at recog.cc:2791 since r14-475-g508f082829af68

2023-05-04 Thread ubizjak at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109733

Uroš Bizjak  changed:

   What|Removed |Added

   Target Milestone|--- |14.0
 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #5 from Uroš Bizjak  ---
Fixed.

[Bug target/109733] [14 Regression] ICE in extract_insn, at recog.cc:2791 since r14-475-g508f082829af68

2023-05-04 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109733

--- Comment #4 from CVS Commits  ---
The master branch has been updated by Uros Bizjak :

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

commit r14-485-g8cac23781753bd8a016507dc9b21ec563e1d9b49
Author: Uros Bizjak 
Date:   Thu May 4 20:26:12 2023 +0200

i386: Tighten ashift to lea splitter operand predicates [PR109733]

The predicates of ashift to lea post-reload splitter were too broad
so the splitter tried to convert the mask shift instruction.  Tighten
operand predicates to match only general registers.

gcc/ChangeLog:

PR target/109733
* config/i386/predicates.md (index_reg_operand): New predicate.
* config/i386/i386.md (ashift to lea spliter): Use
general_reg_operand and index_reg_operand predicates.

[Bug target/109733] [14 Regression] ICE in extract_insn, at recog.cc:2791 since r14-475-g508f082829af68

2023-05-04 Thread ubizjak at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109733

Uroš Bizjak  changed:

   What|Removed |Added

  Attachment #54996|0   |1
is obsolete||

--- Comment #3 from Uroš Bizjak  ---
Created attachment 54997
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=54997=edit
The correct patch.

[Bug target/109733] [14 Regression] ICE in extract_insn, at recog.cc:2791 since r14-475-g508f082829af68

2023-05-04 Thread ubizjak at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109733

Uroš Bizjak  changed:

   What|Removed |Added

   Assignee|unassigned at gcc dot gnu.org  |ubizjak at gmail dot com
 Status|NEW |ASSIGNED

--- Comment #2 from Uroš Bizjak  ---
Created attachment 54996
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=54996=edit
Patch in testing.

[Bug target/109733] [14 Regression] ICE in extract_insn, at recog.cc:2791 since r14-475-g508f082829af68

2023-05-04 Thread ubizjak at gmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109733

--- Comment #1 from Uroš Bizjak  ---
The patched compiler just happens to trigger the existing problem where:

(insn 188 416 379 18 (parallel [
(set (reg:SI 72 k4 [orig:121 _114 ] [121])
(ashift:SI (reg:SI 70 k2 [orig:112 ubound.0 ] [112])
(const_int 2 [0x2])))
(clobber (reg:CC 17 flags))
]) 811 {*ashlsi3_1}
 (nil))

gets split with a post-reload splitter a to non-existent:

(insn 429 416 379 18 (set (reg:SI 72 k4 [orig:121 _114 ] [121])
(mult:SI (reg:SI 70 k2 [orig:112 ubound.0 ] [112])
(const_int 4 [0x4]))) -1
 (nil))

Please note mask registers, which points to a post-reload splitter with too
broad predicates.

[Bug target/109733] [14 Regression] ICE in extract_insn, at recog.cc:2791 since r14-475-g508f082829af68

2023-05-04 Thread marxin at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109733

Martin Liška  changed:

   What|Removed |Added

   Last reconfirmed||2023-05-04
Summary|ICE in extract_insn, at |[14 Regression] ICE in
   |recog.cc:2791 since |extract_insn, at
   |r14-475-g508f082829af68 |recog.cc:2791 since
   ||r14-475-g508f082829af68
 Ever confirmed|0   |1
 Status|UNCONFIRMED |NEW