[PATCH] [clang-cl] Ignore the /o option when /P is specified.
Hans Wennborg
hans at chromium.org
Mon Jun 8 08:29:28 PDT 2015
LGTM with comment.
================
Comment at: test/Driver/cl-outputs.c:266
@@ -265,3 +265,3 @@
// Fio4: "-E"
-// Fio4: "-o" "foo.x"
+// Fio4: "-o" "cl-outputs.i"
----------------
We probably don't need all four test cases for /P with /o anymore.
I think we should also have a note explaining the expected behaviour: that /o doesn't affect /P.
http://reviews.llvm.org/D10313
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the cfe-commits
mailing list