tra added inline comments.

================
Comment at: include/clang/Driver/Options.td:1807
@@ +1806,3 @@
+    "CUDA compilation without --save-temps.">;
+def nostop_on_failure : Flag<["-"], "nostop-on-failure">, 
Flags<[DriverOption]>;
+
----------------
I'd use 'no-' prefix.

================
Comment at: lib/Driver/Driver.cpp:650
@@ -638,3 +649,3 @@
   SmallVector<std::pair<int, const Command *>, 4> FailingCommands;
-  C.ExecuteJobs(C.getJobs(), FailingCommands);
+  C.ExecuteJobs(C.getJobs(), /* StopOnFailure = */ false, FailingCommands);
 
----------------
Why is StopOnFailure is false in this case? Shouldn't it obey command line 
options, too?


http://reviews.llvm.org/D16514



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to