[openssl-commits] [openssl] master update

[Andy Polyakov]

The branch master has been updated
       via  3a1548643f3f9ef6905e31bcde8603a525789abc (commit)
      from  51ac82702dc91cabd3dbf890d8f65b285282c0ce (commit)


- Log -----------------------------------------------------------------
commit 3a1548643f3f9ef6905e31bcde8603a525789abc
Author: Andy Polyakov <appro at openssl.org>
Date:   Mon Sep 18 16:30:28 2017 +0200

    Configure: unify clang's -Qunused-arguments option treatment.
    
    Detect clang even if it's disguised, e.g. cross-compiler or invoked by
    explicit path name, and add the option based on that.
    
    Reviewed-by: Ben Kaduk <kaduk at mit.edu>
    (Merged from https://github.com/openssl/openssl/pull/4383)

-----------------------------------------------------------------------

Summary of changes:
 Configurations/10-main.conf | 4 ++--
 Configure                   | 5 ++++-
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/Configurations/10-main.conf b/Configurations/10-main.conf
index ea1c9ab..ca53937 100644
--- a/Configurations/10-main.conf
+++ b/Configurations/10-main.conf
@@ -763,7 +763,7 @@ sub vms_info {
         inherit_from     => [ "linux-x86" ],
         cc               => "clang",
         cxx              => "clang++",
-        cflags           => add("-Wextra -Qunused-arguments"),
+        cflags           => add("-Wextra"),
     },
     "linux-x86_64" => {
         inherit_from     => [ "linux-generic64", asm("x86_64_asm") ],
@@ -777,7 +777,7 @@ sub vms_info {
         inherit_from     => [ "linux-x86_64" ],
         cc               => "clang",
         cxx              => "clang++",
-        cflags           => add("-Wextra -Qunused-arguments"),
+        cflags           => add("-Wextra"),
     },
     "linux-x32" => {
         inherit_from     => [ "linux-generic32", asm("x86_64_asm") ],
diff --git a/Configure b/Configure
index 4d74b43..326a3ff 100755
--- a/Configure
+++ b/Configure
@@ -141,7 +141,6 @@ my $gcc_devteam_warn = "-DDEBUG_UNUSED"
 #       -Wunused-macros -- no, too tricky for BN and _XOPEN_SOURCE etc
 #       -Wextended-offsetof -- no, needed in CMS ASN1 code
 my $clang_devteam_warn = ""
-        . " -Qunused-arguments"
         . " -Wswitch-default"
         . " -Wno-parentheses-equality"
         . " -Wno-language-extension-token"
@@ -1321,6 +1320,10 @@ if (defined($config{api})) {
     push @{$config{defines}}, $apiflag;
 }
 
+if (defined($predefined{__clang__}) && !$disabled{asm}) {
+    $config{cflags} .= " -Qunused-arguments";
+}
+
 if ($strict_warnings)
 	{
 	my $wopt;