diff options
author | Pierre Moreau <dev@pmoreau.org> | 2020-11-10 11:44:45 +1000 |
---|---|---|
committer | Dave Airlie <airlied@redhat.com> | 2020-11-11 06:30:26 +1000 |
commit | f1b91177fbdc97289e9ddefa8db4b18fdc88f8f0 (patch) | |
tree | 9f4ce4920943ed1cbf5f22b87684128eb178c160 /src/gallium/frontends/clover/spirv/invocation.cpp | |
parent | 330c52476292d2cfba15499f2f74ca224f227260 (diff) |
clover/spirv: avoid strings for version handling
This is extracted from Pierre's WIP versioning patch.
Reviewed-by: Dave Airlie <airlied@redhat.com>
Reviewed-by: Karol Herbst <kherbst@redhat.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/7520>
Diffstat (limited to 'src/gallium/frontends/clover/spirv/invocation.cpp')
-rw-r--r-- | src/gallium/frontends/clover/spirv/invocation.cpp | 36 |
1 files changed, 18 insertions, 18 deletions
diff --git a/src/gallium/frontends/clover/spirv/invocation.cpp b/src/gallium/frontends/clover/spirv/invocation.cpp index 29d7d8f5883..27fa06db46c 100644 --- a/src/gallium/frontends/clover/spirv/invocation.cpp +++ b/src/gallium/frontends/clover/spirv/invocation.cpp @@ -651,19 +651,19 @@ namespace { } spv_target_env - convert_opencl_str_to_target_env(const std::string &opencl_version) { + convert_opencl_version_to_target_env(const cl_version opencl_version) { // Pick 1.2 for 3.0 for now - if (opencl_version == "3.0") { + if (opencl_version == CL_MAKE_VERSION(3, 0, 0)) { return SPV_ENV_OPENCL_1_2; - } else if (opencl_version == "2.2") { + } else if (opencl_version == CL_MAKE_VERSION(2, 2, 0)) { return SPV_ENV_OPENCL_2_2; - } else if (opencl_version == "2.1") { + } else if (opencl_version == CL_MAKE_VERSION(2, 1, 0)) { return SPV_ENV_OPENCL_2_1; - } else if (opencl_version == "2.0") { + } else if (opencl_version == CL_MAKE_VERSION(2, 0, 0)) { return SPV_ENV_OPENCL_2_0; - } else if (opencl_version == "1.2" || - opencl_version == "1.1" || - opencl_version == "1.0") { + } else if (opencl_version == CL_MAKE_VERSION(1, 2, 0) || + opencl_version == CL_MAKE_VERSION(1, 1, 0) || + opencl_version == CL_MAKE_VERSION(1, 0, 0)) { // SPIR-V is only defined for OpenCL >= 1.2, however some drivers // might use it with OpenCL 1.0 and 1.1. return SPV_ENV_OPENCL_1_2; @@ -681,7 +681,7 @@ clover::spirv::compile_program(const std::vector<char> &binary, bool validate) { std::vector<char> source = spirv_to_cpu(binary); - if (validate && !is_valid_spirv(source, dev.device_version_as_string(), r_log)) + if (validate && !is_valid_spirv(source, dev.device_version(), r_log)) throw build_error(); if (!check_capabilities(dev, source, r_log)) @@ -751,9 +751,9 @@ clover::spirv::link_program(const std::vector<module> &modules, std::vector<uint32_t> linked_binary; - const std::string opencl_version = dev.device_version_as_string(); + const cl_version opencl_version = dev.device_version(); const spv_target_env target_env = - convert_opencl_str_to_target_env(opencl_version); + convert_opencl_version_to_target_env(opencl_version); const spvtools::MessageConsumer consumer = validator_consumer; spvtools::Context context(target_env); @@ -771,7 +771,7 @@ clover::spirv::link_program(const std::vector<module> &modules, throw error(CL_LINK_PROGRAM_FAILURE); if (has_flag(llvm::debug::spirv)) - llvm::debug::log(".spvasm", spirv::print_module(final_binary, dev.device_version_as_string())); + llvm::debug::log(".spvasm", spirv::print_module(final_binary, dev.device_version())); for (const auto &mod : modules) m.syms.insert(m.syms.end(), mod.syms.begin(), mod.syms.end()); @@ -783,7 +783,7 @@ clover::spirv::link_program(const std::vector<module> &modules, bool clover::spirv::is_valid_spirv(const std::vector<char> &binary, - const std::string &opencl_version, + const cl_version opencl_version, std::string &r_log) { auto const validator_consumer = [&r_log](spv_message_level_t level, const char *source, @@ -792,7 +792,7 @@ clover::spirv::is_valid_spirv(const std::vector<char> &binary, }; const spv_target_env target_env = - convert_opencl_str_to_target_env(opencl_version); + convert_opencl_version_to_target_env(opencl_version); spvtools::SpirvTools spvTool(target_env); spvTool.SetMessageConsumer(validator_consumer); @@ -802,9 +802,9 @@ clover::spirv::is_valid_spirv(const std::vector<char> &binary, std::string clover::spirv::print_module(const std::vector<char> &binary, - const std::string &opencl_version) { + const cl_version opencl_version) { const spv_target_env target_env = - convert_opencl_str_to_target_env(opencl_version); + convert_opencl_version_to_target_env(opencl_version); spvtools::SpirvTools spvTool(target_env); spv_context spvContext = spvContextCreate(target_env); if (!spvContext) @@ -839,7 +839,7 @@ clover::spirv::supported_versions() { #else bool clover::spirv::is_valid_spirv(const std::vector<char> &/*binary*/, - const std::string &/*opencl_version*/, + const cl_version opencl_version, std::string &/*r_log*/) { return false; } @@ -862,7 +862,7 @@ clover::spirv::link_program(const std::vector<module> &/*modules*/, std::string clover::spirv::print_module(const std::vector<char> &binary, - const std::string &opencl_version) { + const cl_version opencl_version) { return std::string(); } |