[PATCH] D94013: [libclc] Add clspv target for libclc

Jan Vesely via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 14 22:19:08 PST 2021


jvesely added a comment.

> The FMA does pass conformance. I don't have performance numbers for the 64-bit version vs the uint2 version. It would be worse, but the whole  software FMA is already awful performance. The 64-bit version is simpler to understand, so unless you have the same restrictions as I need for clspv, then I wouldn't adopt it.
>
> Hopefully, the different target name will clarify the intent better. This is targeted for use with a cross compiler to Vulkan SPIR-V. The needs are different than the spir targets.

There are existing `spirv` and `spirv64` targets. The new `clspv` target uses the same `spir--` triple as `spirv`, but doesn't use the final spv conversion step and uses O3 <https://reviews.llvm.org/owners/package/3/> optimization. The selection of files is mostly a subset of the existing `spirv` target.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94013



More information about the llvm-commits mailing list