[PATCH] D16784: [OpenMP] Reorganize code to allow specialized code generation for different devices.
Samuel Antao via cfe-commits
cfe-commits at lists.llvm.org
Thu Feb 4 11:48:09 PST 2016
sfantao added inline comments.
================
Comment at: lib/CodeGen/CodeGenModule.cpp:218
@@ -204,1 +217,3 @@
+ }
+ return;
}
----------------
ABataev wrote:
> I think this must be removed
Ok, done.
================
Comment at: lib/Frontend/CompilerInvocation.cpp:1837
@@ +1836,3 @@
+ Diags.Report(clang::diag::err_drv_omp_host_or_device_target_not_supported)
+ << TargetOpts.Triple << Opts.OpenMPIsDevice;
+ break;
----------------
ABataev wrote:
> OpenMPIsDevice is always false here. Do we really need it here?
Ok, hardcoded it to false.
http://reviews.llvm.org/D16784
More information about the cfe-commits
mailing list