[Bug preprocessor/103415] [12 Regression] ICE in cpp_interpret_string_1, at libcpp/charset.c:1739
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103415 Jakub Jelinek changed: What|Removed |Added Resolution|--- |FIXED Status|ASSIGNED|RESOLVED --- Comment #7 from Jakub Jelinek --- Fixed.
[Bug preprocessor/103415] [12 Regression] ICE in cpp_interpret_string_1, at libcpp/charset.c:1739
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103415 --- Comment #6 from CVS Commits --- The master branch has been updated by Jakub Jelinek : https://gcc.gnu.org/g:ac5fd364f0978c62ae759e7b36ce6b912a27546c commit r12-5651-gac5fd364f0978c62ae759e7b36ce6b912a27546c Author: Jakub Jelinek Date: Wed Dec 1 10:07:59 2021 +0100 libcpp: Fix up #__VA_OPT__ handling [PR103415] stringify_arg uses pfile->u_buff to create the string literal. Unfortunately, paste_tokens -> _cpp_lex_direct -> lex_number -> _cpp_unaligned_alloc can in some cases use pfile->u_buff too, which results in losing everything prepared for the string literal until the token pasting. The following patch fixes that by not calling paste_token during the construction of the string literal, but doing that before. All the tokens we are processing have been pushed into a token buffer using tokens_buff_add_token so it is fine if we paste some of them in that buffer (successful pasting creates a new token in that buffer), move following tokens if any to make it contiguous, pop (throw away) the extra tokens at the end and then do stringify_arg. Also, paste_tokens now copies over PREV_WHITE and PREV_FALLTHROUGH flags from the original lhs token to the replacement token. Copying that way the PREV_WHITE flag is needed for the #__VA_OPT__ handling and copying over PREV_FALLTHROUGH fixes the new Wimplicit-fallthrough-38.c test. 2021-12-01 Jakub Jelinek PR preprocessor/103415 libcpp/ * macro.c (stringify_arg): Remove va_opt argument and va_opt handling. (paste_tokens): On successful paste or in PREV_WHITE and PREV_FALLTHROUGH flags from the *plhs token to the new token. (replace_args): Adjust stringify_arg callers. For #__VA_OPT__, perform token pasting in a separate loop before stringify_arg call. gcc/testsuite/ * c-c++-common/cpp/va-opt-8.c: New test. * c-c++-common/Wimplicit-fallthrough-38.c: New test.
[Bug preprocessor/103415] [12 Regression] ICE in cpp_interpret_string_1, at libcpp/charset.c:1739
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103415 Jakub Jelinek changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |jakub at gcc dot gnu.org Status|NEW |ASSIGNED --- Comment #5 from Jakub Jelinek --- Created attachment 51873 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=51873=edit gcc12-pr103415.patch Untested fix.
[Bug preprocessor/103415] [12 Regression] ICE in cpp_interpret_string_1, at libcpp/charset.c:1739
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103415 --- Comment #4 from Jakub Jelinek --- __VA_OPT__ has been supported for a few more years, my change just added support for stringification of __VA_OPT__...
[Bug preprocessor/103415] [12 Regression] ICE in cpp_interpret_string_1, at libcpp/charset.c:1739
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103415 Martin Liška changed: What|Removed |Added CC||jakub at gcc dot gnu.org, ||marxin at gcc dot gnu.org --- Comment #3 from Martin Liška --- (In reply to Andrew Pinski from comment #2) > Confirmed. > > > (In reply to G. Steinmetz from comment #0) > > Started between 20210808 and 20210822 : > > Then it is when __VA_OPT__ support was added in r12-2940-gd56599979211266b. Yes, it started with the revision.
[Bug preprocessor/103415] [12 Regression] ICE in cpp_interpret_string_1, at libcpp/charset.c:1739
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103415 Andrew Pinski changed: What|Removed |Added Ever confirmed|0 |1 Last reconfirmed||2021-11-25 Status|UNCONFIRMED |NEW --- Comment #2 from Andrew Pinski --- Confirmed. (In reply to G. Steinmetz from comment #0) > Started between 20210808 and 20210822 : Then it is when __VA_OPT__ support was added in r12-2940-gd56599979211266b.
[Bug preprocessor/103415] [12 Regression] ICE in cpp_interpret_string_1, at libcpp/charset.c:1739
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103415 Andrew Pinski changed: What|Removed |Added Target Milestone|--- |12.0