[PATCH] D34784: [OpenMP] Add flag for specifying the target device architecture for OpenMP device offloading

[Gheorghe-Teodor Bercea via Phabricator via cfe-commits]

gtbercea added inline comments.


================
Comment at: lib/Driver/ToolChains/Cuda.cpp:478
+    auto MArchList = DAL->getAllArgValues(options::OPT_march_EQ);
+    assert(MArchList.size() < 2 && "At most one GPU arch allowed.");
+    if (MArchList.empty())
----------------
hfinkel wrote:
> Can a user hit this? If so, it must be an actual diagnostic.
A user cannot hit this now, -Xopenmp-target does not lead to duplicate -march flags in DAL anymore.


https://reviews.llvm.org/D34784