Mesa (master): intel/mi_builder: Fix a misleading comment
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Thu Feb 18 21:33:04 UTC 2021
Module: Mesa
Branch: master
Commit: c64d7aa8e3216e5627c41f4fa31ff70c964ca91f
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=c64d7aa8e3216e5627c41f4fa31ff70c964ca91f
Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date: Thu Feb 18 09:37:44 2021 -0600
intel/mi_builder: Fix a misleading comment
This doesn't actually assume the top 32 bits of the source value are
zero. Instead, it does (src >> shift) & UINT32_MAX regardless of what
the top bits of src are.
Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9116>
---
src/intel/common/gen_mi_builder.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/intel/common/gen_mi_builder.h b/src/intel/common/gen_mi_builder.h
index 24e16ac38b4..79a167991e4 100644
--- a/src/intel/common/gen_mi_builder.h
+++ b/src/intel/common/gen_mi_builder.h
@@ -868,7 +868,7 @@ gen_mi_ushr32_imm(struct gen_mi_builder *b,
struct gen_mi_value src, uint32_t shift)
{
/* We right-shift by left-shifting by 32 - shift and taking the top 32 bits
- * of the result. This assumes the top 32 bits are zero.
+ * of the result.
*/
if (shift > 64)
return gen_mi_imm(0);
More information about the mesa-commit
mailing list