[PATCH] D110622: [HIPSPV][3/4] Enable SPIR-V emission for HIP

Wed Dec 8 01:12:18 PST 2021

linjamaki added a comment.

Thanks for the feedback. The `--offload` is meant to support multiple targets but right now it is restricted to one at most. The limitation comes from the HIPActionBuilder and CudaActionBuilderBase which currently expects a single target triple and toolchain for all offload devices. To relax the `--offload` target count cap we would need to adjust the action builders to support multiple target triples and create a separate toolchain for each (unique) target triple.

Details for  the `--offload` option for specifying multiple targets are left open in this patch. What this patch needs is at least an ability to specify a single target (e.g. `--offload=spirv64`).

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110622