Launchpad has imported 7 comments from the remote bug at https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81922.
If you reply to an imported comment from within Launchpad, your comment will be sent to the remote bug automatically. Read more about Launchpad's inter-bugtracker facilities at https://help.launchpad.net/InterBugTracking. ------------------------------------------------------------------------ On 2017-08-22T10:00:52+00:00 Rguenth wrote: Created attachment 42024 unincluded unreduced testcase > g++-7 -std=gnu++1z -S floc.cc floc.cc:187:1: internal compiler error: in output_constructor_regular_field, at varasm.c:5030 continue; ^ Please submit a full bug report, with preprocessed source if appropriate. See <http://bugs.opensuse.org/> for instructions. same ICE with gcc 6 and trunk. Reply at: https://bugs.launchpad.net/ubuntu/+source/gcc-7/+bug/1848795/comments/0 ------------------------------------------------------------------------ On 2017-09-11T19:19:22+00:00 Marxin-m wrote: Confirmed for following reduced test-case: $ cat pr81922.ii struct { char *a; char b[]; } c[]{"", "", {}}; started with r234636. Is the original test-case really valid? Reply at: https://bugs.launchpad.net/ubuntu/+source/gcc-7/+bug/1848795/comments/1 ------------------------------------------------------------------------ On 2017-12-07T15:21:57+00:00 Jakub-gcc wrote: We have a bunch of other PRs for the lack of diagnostics of invalid initialization of flexible array members inside of arrays to non-empty array. We just should diagnose it as invalid. Reply at: https://bugs.launchpad.net/ubuntu/+source/gcc-7/+bug/1848795/comments/2 ------------------------------------------------------------------------ On 2017-12-07T17:03:30+00:00 Jakub-gcc wrote: Created attachment 42811 gcc8-pr81922.patch WIP patch that attempts to handle it like the C FE, will work on testcases tomorrow. Reply at: https://bugs.launchpad.net/ubuntu/+source/gcc-7/+bug/1848795/comments/3 ------------------------------------------------------------------------ On 2017-12-15T20:20:31+00:00 Jakub-gcc wrote: Author: jakub Date: Fri Dec 15 20:20:00 2017 New Revision: 255703 URL: https://gcc.gnu.org/viewcvs?rev=255703&root=gcc&view=rev Log: PR c++/80135 PR c++/81922 * typeck2.c (digest_init_r): Change nested argument type from bool to int. Use code instead of TREE_CODE (type) where possible. If nested == 2, diagnose initialization of flexible array member with STRING_CST. Pass nested to process_init_constructor. Formatting fix. (digest_init, digest_init_flags): Adjust digest_init_r caller. (massage_init_elt): Add nested argument. Pass 2 instead of 1 to digest_init_r's nested argument if nested is non-zero. (process_init_constructor_array): Add nested argument. If nested == 2, diagnose initialization of flexible array member with non-empty braced enclosed list. Pass nested to massage_init_elt. (process_init_constructor_record, process_init_constructor_union): Add nested argument, pass it to massage_init_elt. (process_init_constructor): Add nested argument, pass it to process_init_constructor_{array,record,union}. * init.c (find_field_init): Renamed to ... (find_flexarray_init): ... this. Return NULL_TREE if init is error_mark_node. Don't look through nested CONSTRUCTORs. (warn_placement_new_too_small): Adjust caller. * g++.dg/warn/Wplacement-new-size-1.C (fBx1): Initialize nested flexible array member only with {}. Add dg-warning. (fBx2, fBx3): Remove. * g++.dg/warn/Wplacement-new-size-2.C (fBx1): Initialize nested flexible array member only with {}. Add dg-warning. (fBx2, fBx3): Remove. * g++.dg/warn/Wplacement-new-size-6.C: New test. * g++.dg/ext/flexary13.C (main): Remove test for initialization of nested flexible array member with non-empty initializer. * g++.dg/ext/flexary25.C: New test. * g++.dg/ext/flexary26.C: New test. * g++.dg/ext/flexary27.C: New test. * g++.dg/parse/pr43765.C: Expect diagnostics about initialization of nested flexible array member with non-empty initializer. Expect C++2A diagnostics about mixing of designated and non-designated initializers. Added: trunk/gcc/testsuite/g++.dg/ext/flexary25.C trunk/gcc/testsuite/g++.dg/ext/flexary26.C trunk/gcc/testsuite/g++.dg/ext/flexary27.C trunk/gcc/testsuite/g++.dg/warn/Wplacement-new-size-6.C Modified: trunk/gcc/cp/ChangeLog trunk/gcc/cp/init.c trunk/gcc/cp/typeck2.c trunk/gcc/testsuite/ChangeLog trunk/gcc/testsuite/g++.dg/ext/flexary13.C trunk/gcc/testsuite/g++.dg/parse/pr43765.C trunk/gcc/testsuite/g++.dg/warn/Wplacement-new-size-1.C trunk/gcc/testsuite/g++.dg/warn/Wplacement-new-size-2.C Reply at: https://bugs.launchpad.net/ubuntu/+source/gcc-7/+bug/1848795/comments/4 ------------------------------------------------------------------------ On 2017-12-15T20:22:08+00:00 Jakub-gcc wrote: Fixed on the trunk. Unsure about backports. Reply at: https://bugs.launchpad.net/ubuntu/+source/gcc-7/+bug/1848795/comments/5 ------------------------------------------------------------------------ On 2018-10-26T10:08:57+00:00 Jakub-gcc wrote: GCC 6 branch is being closed Reply at: https://bugs.launchpad.net/ubuntu/+source/gcc-7/+bug/1848795/comments/6 ** Changed in: gcc Status: Unknown => Confirmed ** Changed in: gcc Importance: Unknown => Medium -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1848795 Title: Internal error in varasm.c To manage notifications about this bug go to: https://bugs.launchpad.net/gcc/+bug/1848795/+subscriptions -- ubuntu-bugs mailing list [email protected] https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
