From: Yonggang Luo <luoyonggang@gmail.com>
Date: Sun, 4 Jun 2023 06:42:51 +0000 (+0800)
Subject: util: use uint32_t as the parameter of align function
X-Git-Tag: upstream/23.3.3~7330
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=586391720b31e9589e55c7849c64326a0e2d9208;p=platform%2Fupstream%2Fmesa.git

util: use uint32_t as the parameter of align function

align on negative value doesn't make sense

Reviewed-by: Rohan Garg <rohan.garg@intel.com>
Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/23421>
---

diff --git a/src/amd/compiler/aco_instruction_selection.cpp b/src/amd/compiler/aco_instruction_selection.cpp
index e7c7bb5..9f7a170 100644
--- a/src/amd/compiler/aco_instruction_selection.cpp
+++ b/src/amd/compiler/aco_instruction_selection.cpp
@@ -4010,7 +4010,7 @@ can_use_byte_align_for_global_load(unsigned num_components, unsigned component_s
    if (align_ < 4) {
       assert(component_size < 4);
       unsigned load_size = num_components * component_size;
-      int new_size = align(load_size + (4 - align_), 4);
+      uint32_t new_size = align(load_size + (4 - align_), 4);
       return new_size == align(load_size, 4) && (new_size != 12 || support_12_byte);
    }
    return true;
diff --git a/src/util/u_math.h b/src/util/u_math.h
index 149d223..61c69a2 100644
--- a/src/util/u_math.h
+++ b/src/util/u_math.h
@@ -691,8 +691,8 @@ ROUND_DOWN_TO(uint64_t value, uint32_t alignment)
 /**
  * Align a value, only works pot alignemnts.
  */
-static inline int
-align(int value, int alignment)
+static inline uint32_t
+align(uint32_t value, uint32_t alignment)
 {
    assert(IS_POT(alignment));
    return ALIGN_POT(value, alignment);