https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817
Andrew Pinski changed:
What|Removed |Added
CC||Hi-Angel at yandex dot ru
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817
Eric Gallager changed:
What|Removed |Added
CC||antmak.pub at gmail dot com,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817
Eric Gallager changed:
What|Removed |Added
CC||gcc-bugs at engestrom dot ch
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817
--- Comment #18 from Anton Maklakov ---
Thank you for the answer!
I've got. Using the hint after #endif is not pretty good, by the logic. But
this is easier.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817
--- Comment #17 from Jakub Jelinek ---
(In reply to Anton Maklakov from comment #16)
> Hi,
>
> I have the same problem:
>
>
> int main() {
> int s = 1;
>
> switch (s) {
> case 2:
> s = 2;
> #if B
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817
Anton Maklakov changed:
What|Removed |Added
CC||antmak.pub at gmail dot com
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817
--- Comment #15 from Marc Mutz ---
(In reply to Jakub Jelinek from comment #13)
> (In reply to Marc Mutz from comment #12)
> > Is replacing a matching comment with __attribute__(fallthrough)) so
> > complicated as to make this a wontfix?
>
> It
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817
Eric Gallager changed:
What|Removed |Added
CC||sgunderson at bigfoot dot com
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817
--- Comment #13 from Jakub Jelinek ---
(In reply to Marc Mutz from comment #12)
> Is replacing a matching comment with __attribute__(fallthrough)) so
> complicated as to make this a wontfix?
It is not really possible.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817
--- Comment #12 from Marc Mutz ---
Is replacing a matching comment with __attribute__(fallthrough)) so complicated
as to make this a wontfix?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817
--- Comment #11 from Jakub Jelinek ---
(In reply to Markus Trippelsdorf from comment #10)
> The testcase from PR77955:
>
> markus@x4 /tmp % cat fall.c
> void bar(int);
>
> void foo(int i) {
> switch (i) {
> case 1: {
> bar(1);
> //
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817
--- Comment #10 from Markus Trippelsdorf ---
The testcase from PR77955:
markus@x4 /tmp % cat fall.c
void bar(int);
void foo(int i) {
switch (i) {
case 1: {
bar(1);
// fall-through
}
case 2:
bar(2);
default:
break;
}
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817
Marek Polacek changed:
What|Removed |Added
CC||trippels at gcc dot gnu.org
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817
Marc Mutz changed:
What|Removed |Added
CC||marc.mutz at kdab dot com
--- Comment #8
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817
--- Comment #7 from jim at meyering dot net ---
Thanks for investigating.
I already pushed a workaround for gnulib's vasnprintf problem
(http://git.savannah.gnu.org/cgit/gnulib.git/commit/?id=e63f5eb55570a1ea3e51ce47df33689785e085c1).
I may just
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817
--- Comment #6 from Jakub Jelinek ---
(In reply to Marek Polacek from comment #5)
> Yeah, I think this one's WONTFIX.
Well, in theory we could for the comments that satisfy fallthrough_comment_p
create CPP_COMMENT token and insert it, even
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817
--- Comment #5 from Marek Polacek ---
Yeah, I think this one's WONTFIX.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817
Jakub Jelinek changed:
What|Removed |Added
CC||jakub at gcc dot gnu.org
--- Comment #4
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817
Alan Modra changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817
--- Comment #2 from jim at meyering dot net ---
Oops. That must have been the "worked-around" version.
Swap the #undef and fallthrough comment to repro:
int
foo (int x)
{
switch (x)
{
case 1:
x = 3;
/* fallthrough */
#undef
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817
Marek Polacek changed:
What|Removed |Added
CC||mpolacek at gcc dot gnu.org
--- Comment
21 matches
Mail list logo