[PATCH] D102361: [OpenMP] Add Module metadata for OpenMP compilation

Joseph Huber via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 24 14:45:07 PDT 2021


jhuber6 added inline comments.


================
Comment at: clang/lib/CodeGen/CodeGenModule.cpp:709
+    getModule().addModuleFlag(llvm::Module::Max, "openmp-device",
+                              LangOpts.OpenMP);
+
----------------
jdoerfert wrote:
> Why do we need `OpenMPRuntime` in the conditional?
Probably don't explicitly need it, I'll remove it.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D102361/new/

https://reviews.llvm.org/D102361



More information about the cfe-commits mailing list