make: Makefile.include: Remove -Wno-implicit-fallthrough#9362
make: Makefile.include: Remove -Wno-implicit-fallthrough#9362kaspar030 merged 3 commits intoRIOT-OS:masterfrom
Conversation
|
It is strange, the errors are places with "falls through" statements. Could some workers have a different version than the one running If not, it maybe requires a more safe proof falls through comment. |
|
I really do not get why it is not building… I do not get any error on my computer. |
|
Same for me. Maybe some CI worker is using a different version of the toolchain? Is that possible to check? |
|
This is for mobi3 (failed asymcute_mqttsn on acd52832): |
well, accidentaly that is also the one printing the static test output for the last build. |
|
If you scroll all the way down in the build output, you'll see that the failures are spread across all workers. |
|
So why does it fail? Is it a gcc 5 thing? |
|
Nevermind, I looked at the wrong line. It's gcc7 on arm |
9c69871 to
d77ee6c
Compare
|
After reading #9398 it seems like this error must be the same ccache problem that the clang builds have. The preprocessor is run separately from the compilation stage and therefore the |
|
Setting CCACHE_CPP2=yes fixed the problem with the compiler incorrectly reporting Wimplicit-fallthrough errors for intentional fall through. |
c4c0729 to
dda5573
Compare
Nice to have found the cause, too bad we need that setting also for gcc, as this has non-negligable impact on compilation speed when using CCACHE. |
|
It had a 2:37 minutes impact to do the But, in practice it should be less I think. Now this PR is build with the cache from the other PRs that do not set CCACHE_CPP2. So basically there should be more cache miss than what would be at the end when all PRs use this one and the cache is warm everywhere. |
Well, sometimes correctness should take precedence over speed 🤡 |
|
This should be good to go, right? |
Contribution description
These flags should hopefully not be necessary now.
Issues/PRs references
Introduced in #8603, #8617