[commit: ghc] circleci-centos: rts: Disable fallthrough attribute when compiling with Clang (dc34667)
git at git.haskell.org
git at git.haskell.org
Tue Dec 11 05:28:38 UTC 2018
Repository : ssh://git@git.haskell.org/ghc
On branch : circleci-centos
Link : http://ghc.haskell.org/trac/ghc/changeset/dc346674c4b5609e15560820b33e533e73213b32/ghc
>---------------------------------------------------------------
commit dc346674c4b5609e15560820b33e533e73213b32
Author: Ben Gamari <ben at smart-cactus.org>
Date: Mon Dec 10 21:48:51 2018 -0500
rts: Disable fallthrough attribute when compiling with Clang
Apparently clang doesn't enable implicitly fallthrough warnings by default
http://llvm.org/viewvc/llvm-project?revision=167655&view=revision when compiling
C and the attribute cause warnings of their own (#16019).
>---------------------------------------------------------------
dc346674c4b5609e15560820b33e533e73213b32
includes/Stg.h | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/includes/Stg.h b/includes/Stg.h
index 9b54526..57cb00d 100644
--- a/includes/Stg.h
+++ b/includes/Stg.h
@@ -197,7 +197,11 @@
#endif
/* Used to mark a switch case that falls-through */
-#if (defined(__GNUC__) && __GNUC__ >= 7) || defined(__clang__)
+#if (defined(__GNUC__) && __GNUC__ >= 7)
+// N.B. Don't enable fallthrough annotations when compiling with Clang.
+// Apparently clang doesn't enable implicitly fallthrough warnings by default
+// http://llvm.org/viewvc/llvm-project?revision=167655&view=revision
+// when compiling C and the attribute cause warnings of their own (#16019).
#define FALLTHROUGH GNU_ATTRIBUTE(fallthrough)
#else
#define FALLTHROUGH ((void)0)
More information about the ghc-commits
mailing list