[PATCH] D117137: [Driver] Add CUDA support for --offload param

[Daniele Castagna via Phabricator via cfe-commits]

dcastagna added inline comments.


================
Comment at: clang/lib/Driver/Driver.cpp:153-155
+  if (TT->getArch() == llvm::Triple::spirv64 &&
+      TT->getVendor() == llvm::Triple::UnknownVendor &&
+      TT->getOS() == llvm::Triple::UnknownOS)
----------------
tra wrote:
> What's expected to happen if someone specifies `spirv64-nvidia` ?
> 
> If someone uses `spirv64-foo-bar` I think it would match this condition. 
> 
> Accepting `spirv64-foo-bar`, but not `spirv64-nvidia-unknown` would be somewhat odd. I think we either should check a fully-specified triple, or only check the parts that matter -- `getArch()` in this case, or, maybe, arch and vendor.
This part is the check for the hip offload triple and this patch did not change the logic for HIP (at least not intentionally), it should be the same as the logic specified in the current getHIPOffloadTargetTriple on ToT.
Happy to change it if you think it shuold be different though.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117137