Looks like the core is now -Wunused-value clean with clang.

Another possibility is that clang has become smarter,
and that needs to be made dependent on clang version.
But for now, let's opt for simplicity and less logic.

1 files changed, 0 insertions, 20 deletions

diff --git a/cflags.SH b/cflags.SH
index c166db30e3..5538e92b6d 100755
--- a/cflags.SH
+++ b/cflags.SH
@@ -250,7 +250,6 @@ done
 #
 case "$ccflags$warn" in
 *-Wall*)
-  is_clang=undef
   case "$cc" in
   *g++*)
     # Without -Wno-unused-variable g++ 4.x compiles are rather unwatchable
@@ -271,25 +270,6 @@ case "$ccflags$warn" in
       esac
     done
     ;;
-  *clang*) is_clang=define ;;
-  *) # clang may not be called clang, it may be called cc.
-    case "`$cc -v 2>&1`" in
-    *clang*) is_clang=define ;;
-    esac ;;
-  esac
-
-  case "$is_clang" in
-  define)
-    for f in -Wno-unused-value
-    do
-      case "$warn" in
-      *"$f"*) ;;
-      *)
-        echo "cflags.SH: Adding $f because of clang."
-        warn="$warn $f" ;;
-      esac
-    done
-    ;;
   esac
   ;;
 esac