[PATCH] D67200: Add -static-openmp driver option

Dan Albert via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 5 11:24:12 PDT 2019


danalbert added a comment.

Otherwise LGTM



================
Comment at: clang/lib/Driver/ToolChains/CommonArgs.cpp:503
 bool tools::addOpenMPRuntime(ArgStringList &CmdArgs, const ToolChain &TC,
-                             const ArgList &Args, bool IsOffloadingHost,
-                             bool GompNeedsRT) {
+                             const ArgList &Args, bool StaticHostRuntime,
+                             bool IsOffloadingHost, bool GompNeedsRT) {
----------------
Maybe `ForceStaticHostRuntime`? For configurations where there is only a static runtime available, `StaticHostRuntime = false` won't actually link the dynamic runtime.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67200





More information about the cfe-commits mailing list