[PATCH] D37914: [OpenMP] Don't throw cudalib not found error if only front-end is required.

Artem Belevich via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 15 11:47:58 PDT 2017


tra added inline comments.


================
Comment at: lib/Driver/ToolChains/Cuda.cpp:505-508
+    if ((DeviceOffloadingKind == Action::OFK_OpenMP &&
+         DriverArgs.hasArg(options::OPT_S)) ||
+        (DeviceOffloadingKind == Action::OFK_OpenMP &&
+         DriverArgs.hasArg(options::OPT_c)))
----------------
This could probably be rephrased as:
```
if (DeviceOffloadingKind == Action::OFK_OpenMP && 
    DriverArgs.hasArg(options::OPT_S, options::OPT_c))) ...
```


Repository:
  rL LLVM

https://reviews.llvm.org/D37914





More information about the cfe-commits mailing list