[PATCH] Force an unknown target for the default case to make it work.

Chandler Carruth chandlerc at gmail.com
Thu Feb 5 10:08:37 PST 2015


Looks good with a less overly specific comment (see below)


================
Comment at: test/Driver/clang-exception-flags.cpp:1-3
@@ -1,1 +1,4 @@
-// RUN: %clang -### %s 2>&1 | FileCheck %s -check-prefix=DEFAULT
+// We have to force the target to unknown for the default, since clang
+// might have been built with LLVM_DEFAULT_TARGET_TRIPLE=x86_64-scei-ps4,
+// which defaults to -fno-exceptions.
+// RUN: %clang -### -target unknown %s 2>&1 | FileCheck %s -check-prefix=DEFAULT
----------------
I would just say that the default behavior of exception handling in target specific.

http://reviews.llvm.org/D7422

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list