[PATCH] D77589: libclc: Add Mesa/SPIR-V target

Sat Jul 18 02:42:30 PDT 2020

bbrezillon added inline comments.

================
Comment at: libclc/CMakeLists.txt:286
+			set( opt_flags )
+			set( spvflags --spirv-max-version=1.1 --spirv-ocl-builtins-version=1.2 )
+		else()
----------------
Looks like the `--spirv-ocl-builtins-version` has been renamed to `--spirv-target-env` (see https://github.com/KhronosGroup/SPIRV-LLVM-Translator/commit/667bf137eeda7af93b482552c8959376e9565fc1#diff-8b7efcbcbffdd234c43aeda3bb73eaa5), and AFAICT, this should have been `--spirv-ocl-builtins-version="CL1.2"`.
Maybe we can just drop this option since CL1.2 seems to be the default anyway.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77589