[PATCH] D30087: [Driver] Unify linking of OpenMP runtime

[Pirama Arumuga Nainar via Phabricator via cfe-commits]

pirama added inline comments.


================
Comment at: lib/Driver/Tools.cpp:8683
   if (!Args.hasArg(options::OPT_nostdlib, options::OPT_nodefaultlibs))
-    addOpenMPRuntime(CmdArgs, getToolChain(), Args);
+    addOpenMPRuntime(CmdArgs, getToolChain(), Args, JA);
 
----------------
`addOpenMPRuntime` would add `-lomptarget` even for this call if JA satisfies the newly added check.  That's a deviation from exisiting behavior.  Am I missing some invariant here where `JobAction`s along this path don't have an offloading action?


https://reviews.llvm.org/D30087