https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108701
--- Comment #11 from apjo at tuta dot io ---
(In reply to Andrew Pinski from comment #10)
Okay now that looks like a clang bug. Clang is supposed to diagnose misleading
indentation like that (see: https://godbolt.org/z/1fv4rEseo).
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108701
--- Comment #10 from Andrew Pinski ---
That is clang does not warn about this case here (changed all tabs to spaces to
indepdent of -ftabstop option):
```
int randBytesGet()
{
int t;
while (true) {
if (true)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108701
Andrew Pinski changed:
What|Removed |Added
Status|REOPENED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108701
apjo at tuta dot io changed:
What|Removed |Added
Resolution|INVALID |---
Status|RESOLVED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108701
Andrew Pinski changed:
What|Removed |Added
Resolution|FIXED |INVALID
--- Comment #7 from Andrew Pins
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108701
apjo at tuta dot io changed:
What|Removed |Added
Resolution|INVALID |FIXED
--- Comment #6 from apjo at
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108701
Andrew Pinski changed:
What|Removed |Added
Resolution|FIXED |INVALID
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108701
apjo at tuta dot io changed:
What|Removed |Added
Resolution|INVALID |FIXED
--- Comment #5 from apjo at
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108701
--- Comment #4 from Andrew Pinski ---
Also once you change godbolt to be 8 spaces/tab, it becomes obvious the warning
is correct for the default.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108701
Andrew Pinski changed:
What|Removed |Added
Status|WAITING |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108701
Andrew Pinski changed:
What|Removed |Added
Status|UNCONFIRMED |WAITING
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108701
--- Comment #1 from Andrew Pinski ---
Created attachment 54421
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=54421&action=edit
testcase
12 matches
Mail list logo
