This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
Looks like the core is now -Wunused-value clean with clang.
authorJarkko Hietaniemi <jhi@iki.fi>
Wed, 18 Jun 2014 12:27:54 +0000 (08:27 -0400)
committerJarkko Hietaniemi <jhi@iki.fi>
Wed, 18 Jun 2014 18:14:27 +0000 (14:14 -0400)
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.

cflags.SH

index c166db3..5538e92 100755 (executable)
--- 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