From 2b62d7c631d82f01dfe73b857033a79c60dd88ea Mon Sep 17 00:00:00 2001 From: Lachlan Deakin Date: Fri, 18 Oct 2024 16:58:19 +1100 Subject: [PATCH] Rename signed `get_constant_*` functions --- vulkano/src/pipeline/shader/validate_runtime.rs | 11 ++++++----- vulkano/src/shader/reflect.rs | 7 +++++-- 2 files changed, 11 insertions(+), 7 deletions(-) diff --git a/vulkano/src/pipeline/shader/validate_runtime.rs b/vulkano/src/pipeline/shader/validate_runtime.rs index d6e90abe68..bd23fd8f2a 100644 --- a/vulkano/src/pipeline/shader/validate_runtime.rs +++ b/vulkano/src/pipeline/shader/validate_runtime.rs @@ -6,8 +6,9 @@ use crate::{ }, shader::{ reflect::{ - get_constant, get_constant_composite, get_constant_composite_composite, - get_constant_float_composite, get_constant_maybe_composite, size_of_type, + get_constant, get_constant_composite, get_constant_float_composite, + get_constant_signed_composite_composite, get_constant_signed_maybe_composite, + size_of_type, }, spirv::{ Capability, Decoration, Dim, ExecutionMode, ExecutionModel, FunctionInfo, Id, @@ -2468,7 +2469,7 @@ impl<'a> RuntimeValidator<'a> { if let Some(components) = image_operands .const_offset .or(image_operands.offset) - .and_then(|offset| get_constant_maybe_composite(self.spirv, offset)) + .and_then(|offset| get_constant_signed_maybe_composite(self.spirv, offset)) { for offset in components { if offset < properties.min_texel_gather_offset as i64 { @@ -2497,7 +2498,7 @@ impl<'a> RuntimeValidator<'a> { } } else if let Some(elements) = image_operands .const_offsets - .and_then(|id| get_constant_composite_composite(self.spirv, id)) + .and_then(|id| get_constant_signed_composite_composite(self.spirv, id)) { for components in elements { for offset in components { @@ -2534,7 +2535,7 @@ impl<'a> RuntimeValidator<'a> { if let Some(image_operands) = instruction.image_operands() { if let Some(components) = image_operands .const_offset - .and_then(|offset| get_constant_maybe_composite(self.spirv, offset)) + .and_then(|offset| get_constant_signed_maybe_composite(self.spirv, offset)) { for offset in components { if offset < properties.min_texel_offset as i64 { diff --git a/vulkano/src/shader/reflect.rs b/vulkano/src/shader/reflect.rs index 21804139b9..3a236306b3 100644 --- a/vulkano/src/shader/reflect.rs +++ b/vulkano/src/shader/reflect.rs @@ -1242,7 +1242,10 @@ fn integer_constant_to_i64(spirv: &Spirv, value: &[u32], result_type_id: Id) -> } } -pub(crate) fn get_constant_maybe_composite(spirv: &Spirv, id: Id) -> Option> { +pub(crate) fn get_constant_signed_maybe_composite( + spirv: &Spirv, + id: Id, +) -> Option> { match spirv.id(id).instruction() { Instruction::Constant { value, @@ -1270,7 +1273,7 @@ pub(crate) fn get_constant_maybe_composite(spirv: &Spirv, id: Id) -> Option Option; 4]>> {