[PATCH] D112410: [SPIR-V] Add a toolchain for SPIR-V in clang
Anastasia Stulova via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Dec 10 11:01:08 PST 2021
Anastasia added inline comments.
================
Comment at: clang/docs/UsersManual.rst:3559
+by the OpenCL driver that support SPIR-V consumption or it can be compiled further
+by offline SPIR-V consumer tools.
+
----------------
Btw do we want to say anything about the limitations of translating IR when optimizations are enabled?
================
Comment at: clang/test/Driver/spirv-toolchain.cl:10
+// SPV64-SAME: "-o" [[BC:".*bc"]]
+// SPV64: {{".*llvm-spirv.*"}} [[BC]] "-o" {{".*o"}}
+
----------------
svenvh wrote:
> Any reason to not just check for `llvm-spirv{{.*}}`, for consistency with the clang check above?
Good question, apparently some tools get some target prefixes like if you look at `Driver::generatePrefixedToolNames`:
https://clang.llvm.org/doxygen/Driver_8cpp_source.html#l05169
But perhaps it doesn't happen for `llvm-spirv` and we can safely omit the prefix?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D112410/new/
https://reviews.llvm.org/D112410
More information about the cfe-commits
mailing list