ABataev added inline comments.

================
Comment at: lib/Frontend/CompilerInvocation.cpp:2167-2172
+    // Set the flag to prevent the implementation from emitting device 
exception
+    // handling code for those requiring so.
+    if (Opts.OpenMPIsDevice && T.isNVPTX()) {
+      Opts.Exceptions = 0;
+      Opts.CXXExceptions = 0;
+    }
----------------
I'm not sure this is the right place for this code.


================
Comment at: test/OpenMP/target_parallel_no_exceptions.cpp:4
+
+#include <stdio.h>
+
----------------
You can't use `#include` directive in tests, declare `printf` function 
yourself, if you really need it.


Repository:
  rL LLVM

https://reviews.llvm.org/D29904



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

Reply via email to