public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug target/101322] ICE in copy_to_mode_reg, at explow.c:651
       [not found] <bug-101322-4@http.gcc.gnu.org/bugzilla/>
@ 2022-08-26 19:15 ` bergner at gcc dot gnu.org
  2022-09-01  2:17 ` cvs-commit at gcc dot gnu.org
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 6+ messages in thread
From: bergner at gcc dot gnu.org @ 2022-08-26 19:15 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101322

Peter Bergner <bergner at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Last reconfirmed|                            |2022-08-26
             Target|powerpc-*-linux-gnu         |powerpc*-*-linux-gnu
             Status|UNCONFIRMED                 |ASSIGNED
     Ever confirmed|0                           |1
           Assignee|unassigned at gcc dot gnu.org      |bergner at gcc dot gnu.org

--- Comment #1 from Peter Bergner <bergner at gcc dot gnu.org> ---
Confirmed.  I see what the problem is.  When we expand the built-in, the cast
to the proper type is lost and we only see a char *vpp.  We then use that type
to create a temporary and we end up with a reg:QI rather than a reg:OO.

Is the casted pointer type supposed to be stripped away like that?

Anyway, I can work around that since I know what the temporary type should be
rather than stealing the type from the src operand.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug target/101322] ICE in copy_to_mode_reg, at explow.c:651
       [not found] <bug-101322-4@http.gcc.gnu.org/bugzilla/>
  2022-08-26 19:15 ` [Bug target/101322] ICE in copy_to_mode_reg, at explow.c:651 bergner at gcc dot gnu.org
@ 2022-09-01  2:17 ` cvs-commit at gcc dot gnu.org
  2022-09-03  1:50 ` cvs-commit at gcc dot gnu.org
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 6+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2022-09-01  2:17 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101322

--- Comment #2 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Peter Bergner <bergner@gcc.gnu.org>:

https://gcc.gnu.org/g:2985049049f12b0aa3366ca244d387820385b9e8

commit r13-2331-g2985049049f12b0aa3366ca244d387820385b9e8
Author: Peter Bergner <bergner@linux.ibm.com>
Date:   Wed Aug 31 21:14:36 2022 -0500

    rs6000: Don't ICE when we disassemble an MMA variable [PR101322]

    When we expand an MMA disassemble built-in with C++ using a pointer that
    is cast to a valid MMA type, the type isn't passed down to the expand
    machinery and we end up using the base type of the pointer which leads to
    an ICE.  This patch enforces we always use the correct MMA type regardless
    of the pointer type being used.

    2022-08-31  Peter Bergner  <bergner@linux.ibm.com>

    gcc/
            PR target/101322
            * config/rs6000/rs6000-builtin.cc (rs6000_gimple_fold_mma_builtin):
            Enforce the use of a valid MMA pointer type.

    gcc/testsuite/
            PR target/101322
            * g++.target/powerpc/pr101322.C: New test.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug target/101322] ICE in copy_to_mode_reg, at explow.c:651
       [not found] <bug-101322-4@http.gcc.gnu.org/bugzilla/>
  2022-08-26 19:15 ` [Bug target/101322] ICE in copy_to_mode_reg, at explow.c:651 bergner at gcc dot gnu.org
  2022-09-01  2:17 ` cvs-commit at gcc dot gnu.org
@ 2022-09-03  1:50 ` cvs-commit at gcc dot gnu.org
  2022-09-12 17:40 ` cvs-commit at gcc dot gnu.org
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 6+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2022-09-03  1:50 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101322

--- Comment #3 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-12 branch has been updated by Peter Bergner
<bergner@gcc.gnu.org>:

https://gcc.gnu.org/g:2d4f60f206cf1100b7484d708f6c913762618676

commit r12-8740-g2d4f60f206cf1100b7484d708f6c913762618676
Author: Peter Bergner <bergner@linux.ibm.com>
Date:   Wed Aug 31 21:14:36 2022 -0500

    rs6000: Don't ICE when we disassemble an MMA variable [PR101322]

    When we expand an MMA disassemble built-in with C++ using a pointer that
    is cast to a valid MMA type, the type isn't passed down to the expand
    machinery and we end up using the base type of the pointer which leads to
    an ICE.  This patch enforces we always use the correct MMA type regardless
    of the pointer type being used.

    2022-08-31  Peter Bergner  <bergner@linux.ibm.com>

    gcc/
            PR target/101322
            * config/rs6000/rs6000-builtin.cc (rs6000_gimple_fold_mma_builtin):
            Enforce the use of a valid MMA pointer type.

    gcc/testsuite/
            PR target/101322
            * g++.target/powerpc/pr101322.C: New test.

    (cherry picked from commit 2985049049f12b0aa3366ca244d387820385b9e8)

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug target/101322] ICE in copy_to_mode_reg, at explow.c:651
       [not found] <bug-101322-4@http.gcc.gnu.org/bugzilla/>
                   ` (2 preceding siblings ...)
  2022-09-03  1:50 ` cvs-commit at gcc dot gnu.org
@ 2022-09-12 17:40 ` cvs-commit at gcc dot gnu.org
  2022-09-13 22:26 ` cvs-commit at gcc dot gnu.org
  2022-09-13 22:27 ` bergner at gcc dot gnu.org
  5 siblings, 0 replies; 6+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2022-09-12 17:40 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101322

--- Comment #4 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-11 branch has been updated by Peter Bergner
<bergner@gcc.gnu.org>:

https://gcc.gnu.org/g:15ae5487dfd3f437b47a26d6198c8e1257fec2fc

commit r11-10251-g15ae5487dfd3f437b47a26d6198c8e1257fec2fc
Author: Peter Bergner <bergner@linux.ibm.com>
Date:   Wed Aug 31 21:14:36 2022 -0500

    rs6000: Don't ICE when we disassemble an MMA variable [PR101322]

    When we expand an MMA disassemble built-in with C++ using a pointer that
    is cast to a valid MMA type, the type isn't passed down to the expand
    machinery and we end up using the base type of the pointer which leads to
    an ICE.  This patch enforces we always use the correct MMA type regardless
    of the pointer type being used.

    2022-08-31  Peter Bergner  <bergner@linux.ibm.com>

    gcc/
            PR target/101322
            * config/rs6000/rs6000-call.c (rs6000_gimple_fold_mma_builtin):
            Enforce the use of a valid MMA pointer type.

    gcc/testsuite/
            PR target/101322
            * g++.target/powerpc/pr101322.C: New test.

    (cherry picked from commit 2985049049f12b0aa3366ca244d387820385b9e8)

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug target/101322] ICE in copy_to_mode_reg, at explow.c:651
       [not found] <bug-101322-4@http.gcc.gnu.org/bugzilla/>
                   ` (3 preceding siblings ...)
  2022-09-12 17:40 ` cvs-commit at gcc dot gnu.org
@ 2022-09-13 22:26 ` cvs-commit at gcc dot gnu.org
  2022-09-13 22:27 ` bergner at gcc dot gnu.org
  5 siblings, 0 replies; 6+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2022-09-13 22:26 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101322

--- Comment #5 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-10 branch has been updated by Peter Bergner
<bergner@gcc.gnu.org>:

https://gcc.gnu.org/g:8301b8e84d833467b435b33592e0aa599d2db582

commit r10-10983-g8301b8e84d833467b435b33592e0aa599d2db582
Author: Peter Bergner <bergner@linux.ibm.com>
Date:   Wed Aug 31 21:14:36 2022 -0500

    rs6000: Don't ICE when we disassemble an MMA variable [PR101322]

    When we expand an MMA disassemble built-in with C++ using a pointer that
    is cast to a valid MMA type, the type isn't passed down to the expand
    machinery and we end up using the base type of the pointer which leads to
    an ICE.  This patch enforces we always use the correct MMA type regardless
    of the pointer type being used.

    2022-08-31  Peter Bergner  <bergner@linux.ibm.com>

    gcc/
            PR target/101322
            * config/rs6000/rs6000-call.c (rs6000_gimple_fold_mma_builtin):
            Enforce the use of a valid MMA pointer type.

    gcc/testsuite/
            PR target/101322
            * g++.target/powerpc/pr101322.C: New test.

    (cherry picked from commit 2985049049f12b0aa3366ca244d387820385b9e8)

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug target/101322] ICE in copy_to_mode_reg, at explow.c:651
       [not found] <bug-101322-4@http.gcc.gnu.org/bugzilla/>
                   ` (4 preceding siblings ...)
  2022-09-13 22:26 ` cvs-commit at gcc dot gnu.org
@ 2022-09-13 22:27 ` bergner at gcc dot gnu.org
  5 siblings, 0 replies; 6+ messages in thread
From: bergner at gcc dot gnu.org @ 2022-09-13 22:27 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101322

Peter Bergner <bergner at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #6 from Peter Bergner <bergner at gcc dot gnu.org> ---
Fixed everywhere.

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2022-09-13 22:27 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <bug-101322-4@http.gcc.gnu.org/bugzilla/>
2022-08-26 19:15 ` [Bug target/101322] ICE in copy_to_mode_reg, at explow.c:651 bergner at gcc dot gnu.org
2022-09-01  2:17 ` cvs-commit at gcc dot gnu.org
2022-09-03  1:50 ` cvs-commit at gcc dot gnu.org
2022-09-12 17:40 ` cvs-commit at gcc dot gnu.org
2022-09-13 22:26 ` cvs-commit at gcc dot gnu.org
2022-09-13 22:27 ` bergner at gcc dot gnu.org

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).