public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug target/87496] ICE in aggregate_value_p at gcc/function.c:2046
       [not found] <bug-87496-4@http.gcc.gnu.org/bugzilla/>
@ 2022-03-04 15:04 ` cvs-commit at gcc dot gnu.org
  2022-03-18 19:10 ` cvs-commit at gcc dot gnu.org
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 5+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2022-03-04 15:04 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #14 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:cb16bc3b5f34733ef9bbf8d2e3acacdecb099a62

commit r12-7485-gcb16bc3b5f34733ef9bbf8d2e3acacdecb099a62
Author: Peter Bergner <bergner@linux.ibm.com>
Date:   Fri Mar 4 09:03:44 2022 -0600

    rs6000: Allow -mlong-double-64 after -mabi={ibm,ieee}longdouble [PR104208,
PR87496]

    The glibc build is showing a build error due to extra "error" checking from
my
    PR87496 fix.  That checking was overeager, disallowing setting the long
double
    size to 64-bits if the 128-bit long double ABI had already been specified.
    Now we only emit an error if we specify a 128-bit long double ABI if our
    long double size is not 128 bits.  This also fixes an erroneous error when
    -mabi=ieeelongdouble is used and ISA 2.06 is not enabled, but the long
double
    size has been changed to 64 bits.

    2022-03-04  Peter Bergner  <bergner@linux.ibm.com>

    gcc/
            PR target/87496
            PR target/104208
            * config/rs6000/rs6000.cc (rs6000_option_override_internal): Make
the
            ISA 2.06 requirement for -mabi=ieeelongdouble conditional on
            -mlong-double-128.
            Move the -mabi=ieeelongdouble and -mabi=ibmlongdouble error
checking
            from here...
            * common/config/rs6000/rs6000-common.cc (rs6000_handle_option):
            ... to here.

    gcc/testsuite/
            PR target/87496
            PR target/104208
            * gcc.target/powerpc/pr104208-1.c: New test.
            * gcc.target/powerpc/pr104208-2.c: Likewise.
            * gcc.target/powerpc/pr87496-2.c: Swap long double options to
trigger
            the expected error.
            * gcc.target/powerpc/pr87496-3.c: Likewise.

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

* [Bug target/87496] ICE in aggregate_value_p at gcc/function.c:2046
       [not found] <bug-87496-4@http.gcc.gnu.org/bugzilla/>
  2022-03-04 15:04 ` [Bug target/87496] ICE in aggregate_value_p at gcc/function.c:2046 cvs-commit at gcc dot gnu.org
@ 2022-03-18 19:10 ` cvs-commit at gcc dot gnu.org
  2022-03-18 19:10 ` cvs-commit at gcc dot gnu.org
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 5+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2022-03-18 19:10 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #15 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:51a34fdff2af99e2bb096436d6872c8ea434d177

commit r11-9668-g51a34fdff2af99e2bb096436d6872c8ea434d177
Author: Peter Bergner <bergner@linux.ibm.com>
Date:   Fri Mar 18 14:08:44 2022 -0500

    rs6000: Allow -mlong-double-64 after -mabi={ibm,ieee}longdouble [PR104208,
PR87496]

    The glibc build is showing a build error due to extra "error" checking from
my
    PR87496 fix.  That checking was overeager, disallowing setting the long
double
    size to 64-bits if the 128-bit long double ABI had already been specified.
    Now we only emit an error if we specify a 128-bit long double ABI if our
    long double size is not 128 bits.  This also fixes an erroneous error when
    -mabi=ieeelongdouble is used and ISA 2.06 is not enabled, but the long
double
    size has been changed to 64 bits.

    2022-03-04  Peter Bergner  <bergner@linux.ibm.com>

    gcc/
            PR target/87496
            PR target/104208
            * config/rs6000/rs6000.c (rs6000_option_override_internal): Make
the
            ISA 2.06 requirement for -mabi=ieeelongdouble conditional on
            -mlong-double-128.
            Move the -mabi=ieeelongdouble and -mabi=ibmlongdouble error
checking
            from here...
            * common/config/rs6000/rs6000-common.c (rs6000_handle_option):
            ... to here.

    gcc/testsuite/
            PR target/87496
            PR target/104208
            * gcc.target/powerpc/pr104208-1.c: New test.
            * gcc.target/powerpc/pr104208-2.c: Likewise.
            * gcc.target/powerpc/pr87496-2.c: Swap long double options to
trigger
            the expected error.
            * gcc.target/powerpc/pr87496-3.c: Likewise.

    (cherry picked from commit cb16bc3b5f34733ef9bbf8d2e3acacdecb099a62)

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

* [Bug target/87496] ICE in aggregate_value_p at gcc/function.c:2046
       [not found] <bug-87496-4@http.gcc.gnu.org/bugzilla/>
  2022-03-04 15:04 ` [Bug target/87496] ICE in aggregate_value_p at gcc/function.c:2046 cvs-commit at gcc dot gnu.org
  2022-03-18 19:10 ` cvs-commit at gcc dot gnu.org
@ 2022-03-18 19:10 ` cvs-commit at gcc dot gnu.org
  2023-05-16  5:37 ` cvs-commit at gcc dot gnu.org
  2023-05-16  5:47 ` pault at gcc dot gnu.org
  4 siblings, 0 replies; 5+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2022-03-18 19:10 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #16 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:063ef668c716ae0bf9c0f79ae62243e89075b475

commit r10-10504-g063ef668c716ae0bf9c0f79ae62243e89075b475
Author: Peter Bergner <bergner@linux.ibm.com>
Date:   Fri Mar 18 14:09:26 2022 -0500

    rs6000: Allow -mlong-double-64 after -mabi={ibm,ieee}longdouble [PR104208,
PR87496]

    The glibc build is showing a build error due to extra "error" checking from
my
    PR87496 fix.  That checking was overeager, disallowing setting the long
double
    size to 64-bits if the 128-bit long double ABI had already been specified.
    Now we only emit an error if we specify a 128-bit long double ABI if our
    long double size is not 128 bits.  This also fixes an erroneous error when
    -mabi=ieeelongdouble is used and ISA 2.06 is not enabled, but the long
double
    size has been changed to 64 bits.

    2022-03-04  Peter Bergner  <bergner@linux.ibm.com>

    gcc/
            PR target/87496
            PR target/104208
            * config/rs6000/rs6000.c (rs6000_option_override_internal): Make
the
            ISA 2.06 requirement for -mabi=ieeelongdouble conditional on
            -mlong-double-128.
            Move the -mabi=ieeelongdouble and -mabi=ibmlongdouble error
checking
            from here...
            * common/config/rs6000/rs6000-common.c (rs6000_handle_option):
            ... to here.

    gcc/testsuite/
            PR target/87496
            PR target/104208
            * gcc.target/powerpc/pr104208-1.c: New test.
            * gcc.target/powerpc/pr104208-2.c: Likewise.
            * gcc.target/powerpc/pr87496-2.c: Swap long double options to
trigger
            the expected error.
            * gcc.target/powerpc/pr87496-3.c: Likewise.

    (cherry picked from commit cb16bc3b5f34733ef9bbf8d2e3acacdecb099a62)

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

* [Bug target/87496] ICE in aggregate_value_p at gcc/function.c:2046
       [not found] <bug-87496-4@http.gcc.gnu.org/bugzilla/>
                   ` (2 preceding siblings ...)
  2022-03-18 19:10 ` cvs-commit at gcc dot gnu.org
@ 2023-05-16  5:37 ` cvs-commit at gcc dot gnu.org
  2023-05-16  5:47 ` pault at gcc dot gnu.org
  4 siblings, 0 replies; 5+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2023-05-16  5:37 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #17 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Paul Thomas <pault@gcc.gnu.org>:

https://gcc.gnu.org/g:6c95fe9bc0553743098eeaa739f14b885050fa42

commit r14-870-g6c95fe9bc0553743098eeaa739f14b885050fa42
Author: Paul Thomas <pault@gcc.gnu.org>
Date:   Tue May 16 06:35:40 2023 +0100

    Fortran: Fix an assortment of bugs

    2023-05-16  Paul Thomas  <pault@gcc.gnu.org>

    gcc/fortran
            PR fortran/105152
            * interface.cc (gfc_compare_actual_formal): Emit an error if an
            unlimited polymorphic actual is not matched either to an
            unlimited or assumed type formal argument.

            PR fortran/100193
            * resolve.cc (resolve_ordinary_assign): Emit an error if the
            var expression of an ordinary assignment is a proc pointer
            component.

            PR fortran/87496
            * trans-array.cc (gfc_walk_array_ref): Provide assumed shape
            arrays coming from interface mapping with a viable arrayspec.

            PR fortran/103389
            * trans-expr.cc (gfc_conv_intrinsic_to_class): Tidy up flagging
            of unlimited polymorphic 'class_ts'.
            (gfc_conv_gfc_desc_to_cfi_desc): Assumed type is unlimited
            polymorphic and should accept any actual type.

            PR fortran/104429
            (gfc_conv_procedure_call): Replace dreadful kludge with a call
            to gfc_finalize_tree_expr. Avoid dereferencing a void pointer
            by giving it the pointer type of the actual argument.

            PR fortran/82774
            (alloc_scalar_allocatable_subcomponent): Shorten the function
            name and replace the symbol argument with the se string length.
            If a deferred length character length is either not present or
            is not a variable, give the typespec a variable and assign the
            string length to that. Use gfc_deferred_strlen to find the
            hidden string length component.
            (gfc_trans_subcomponent_assign): Convert the expression before
            the call to alloc_scalar_allocatable_subcomponent so that a
            good string length is provided.
            (gfc_trans_structure_assign): Remove the unneeded derived type
            symbol from calls to gfc_trans_subcomponent_assign.

    gcc/testsuite/
            PR fortran/105152
            * gfortran.dg/pr105152.f90 : New test

            PR fortran/100193
            * gfortran.dg/pr100193.f90 : New test

            PR fortran/87946
            * gfortran.dg/pr87946.f90 : New test

            PR fortran/103389
            * gfortran.dg/pr103389.f90 : New test

            PR fortran/104429
            * gfortran.dg/pr104429.f90 : New test

            PR fortran/82774
            * gfortran.dg/pr82774.f90 : New test

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

* [Bug target/87496] ICE in aggregate_value_p at gcc/function.c:2046
       [not found] <bug-87496-4@http.gcc.gnu.org/bugzilla/>
                   ` (3 preceding siblings ...)
  2023-05-16  5:37 ` cvs-commit at gcc dot gnu.org
@ 2023-05-16  5:47 ` pault at gcc dot gnu.org
  4 siblings, 0 replies; 5+ messages in thread
From: pault at gcc dot gnu.org @ 2023-05-16  5:47 UTC (permalink / raw)
  To: gcc-bugs

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

Paul Thomas <pault at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |pault at gcc dot gnu.org

--- Comment #18 from Paul Thomas <pault at gcc dot gnu.org> ---
Sorry for the noise - a bit of a slip occurred in the ChangeLog.

Cheers

Paul

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

end of thread, other threads:[~2023-05-16  5:47 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <bug-87496-4@http.gcc.gnu.org/bugzilla/>
2022-03-04 15:04 ` [Bug target/87496] ICE in aggregate_value_p at gcc/function.c:2046 cvs-commit at gcc dot gnu.org
2022-03-18 19:10 ` cvs-commit at gcc dot gnu.org
2022-03-18 19:10 ` cvs-commit at gcc dot gnu.org
2023-05-16  5:37 ` cvs-commit at gcc dot gnu.org
2023-05-16  5:47 ` pault 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).