Revert r340552, "Driver: Enable address-significance tables by default when targeting...
authorPeter Collingbourne <peter@pcc.me.uk>
Thu, 23 Aug 2018 21:34:57 +0000 (21:34 +0000)
committerPeter Collingbourne <peter@pcc.me.uk>
Thu, 23 Aug 2018 21:34:57 +0000 (21:34 +0000)
Received multiple reports of breakage due to undefined symbols
suspected to be caused by this change.

llvm-svn: 340579

clang/lib/Driver/ToolChains/Clang.cpp
clang/test/Driver/addrsig.c

index 8d73db1..45c63d5 100644 (file)
@@ -4857,8 +4857,7 @@ void Clang::ConstructJob(Compilation &C, const JobAction &JA,
   }
 
   if (Args.hasFlag(options::OPT_faddrsig, options::OPT_fno_addrsig,
-                   (getToolChain().getTriple().isOSBinFormatELF() ||
-                    getToolChain().getTriple().isOSBinFormatCOFF()) &&
+                   getToolChain().getTriple().isOSBinFormatELF() &&
                        getToolChain().useIntegratedAs()))
     CmdArgs.push_back("-faddrsig");
 
index 4eea415..fd5598c 100644 (file)
@@ -1,5 +1,4 @@
 // RUN: %clang -### -target x86_64-unknown-linux -c %s 2>&1 | FileCheck -check-prefix=ADDRSIG %s
-// RUN: %clang -### -target x86_64-pc-win32 -c %s 2>&1 | FileCheck -check-prefix=ADDRSIG %s
 // RUN: %clang -### -target x86_64-unknown-linux -fno-integrated-as -c %s 2>&1 | FileCheck -check-prefix=NO-ADDRSIG %s
 // RUN: %clang -### -target x86_64-unknown-linux -fno-integrated-as -faddrsig -c %s 2>&1 | FileCheck -check-prefix=ADDRSIG %s
 // RUN: %clang -### -target x86_64-unknown-linux -fno-addrsig -c %s 2>&1 | FileCheck -check-prefix=NO-ADDRSIG %s