]> granicus.if.org Git - clang/commitdiff
clang-cl: Enable -fexceptions but not -fcxx-exceptions by default
authorReid Kleckner <reid@kleckner.net>
Fri, 30 Jan 2015 01:04:16 +0000 (01:04 +0000)
committerReid Kleckner <reid@kleckner.net>
Fri, 30 Jan 2015 01:04:16 +0000 (01:04 +0000)
This enables proper IRgen of SEH constructs.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@227528 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Driver/Tools.cpp
test/Driver/cl-eh.cpp

index 91cf841577b53789eb60c439e56aae990c81528f..84c3228f6d286998b07423c956249d500406aa91 100644 (file)
@@ -4820,10 +4820,10 @@ void Clang::AddClangCLArgs(const ArgList &Args, ArgStringList &CmdArgs) const {
   const Driver &D = getToolChain().getDriver();
   EHFlags EH = parseClangCLEHFlags(D, Args);
   // FIXME: Do something with NoExceptC.
-  if (EH.Synch || EH.Asynch) {
-    CmdArgs.push_back("-fexceptions");
+  if (EH.Synch || EH.Asynch)
     CmdArgs.push_back("-fcxx-exceptions");
-  }
+  // Always add -fexceptions to allow SEH __try.
+  CmdArgs.push_back("-fexceptions");
 
   // /EP should expand to -E -P.
   if (Args.hasArg(options::OPT__SLASH_EP)) {
index 8a3450a88c063972945398a88b64ec6d188a44f4..c9cb90350b2fbdeaf2d1b72ff5e202549ee0c193 100644 (file)
@@ -5,18 +5,23 @@
 // command-line option, e.g. on Mac where %s is commonly under /Users.
 
 // RUN: %clang_cl /c /EHsc -### -- %s 2>&1 | FileCheck -check-prefix=EHsc %s
+// EHsc: "-fcxx-exceptions"
 // EHsc: "-fexceptions"
 
 // RUN: %clang_cl /c /EHs-c- -### -- %s 2>&1 | FileCheck -check-prefix=EHs_c_ %s
-// EHs_c_-NOT: "-fexceptions"
+// EHs_c_-NOT: "-fcxx-exceptions"
+// EHs_c_: "-fexceptions"
 
 // RUN: %clang_cl /c /EHs- /EHc- -### -- %s 2>&1 | FileCheck -check-prefix=EHs_EHc_ %s
-// EHs_EHc_-NOT: "-fexceptions"
+// EHs_EHc_-NOT: "-fcxx-exceptions"
+// EHs_EHc_: "-fexceptions"
 
 // RUN: %clang_cl /c /EHs- /EHs -### -- %s 2>&1 | FileCheck -check-prefix=EHs_EHs %s
+// EHs_EHs: "-fcxx-exceptions"
 // EHs_EHs: "-fexceptions"
 
 // RUN: %clang_cl /c /EHs- /EHsa -### -- %s 2>&1 | FileCheck -check-prefix=EHs_EHa %s
+// EHs_EHa: "-fcxx-exceptions"
 // EHs_EHa: "-fexceptions"
 
 // RUN: %clang_cl /c /EHinvalid -### -- %s 2>&1 | FileCheck -check-prefix=EHinvalid %s