[PATCH] D29904: [OpenMP] Prevent emission of exception handling code when using OpenMP to offload to NVIDIA devices.

[Alexey Bataev via Phabricator via cfe-commits]

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