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

Tue Jan 25 06:22:50 PST 2022

yaxunl added a comment.

In D117137#3268548 <https://reviews.llvm.org/D117137#3268548>, @linjamaki wrote:

> SPIR-V target requires that the OS and the environment type is unknown (see TargetInfo::AllocateTarget and BaseSPIRTargetInfo). The clang would fail to create a SPIR-V target if there is an OS or environment component in the target string known by the Triple. This could lead to a misleading error message.

Does that mean only "spirv{64}-unkown-unkown" is acceptable, or "spirv{64}-amd-unkown-unknown" is also acceptable?

One usage of vendor component in spirv triple is that it may be used to choose toolchain if there are multiple toolchains supporting spirv.

@mkuper What are intended use of OS and environment components of spirv triple?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117137