Mesa (master): clover: fix getting scalar args api size
Serge Martin
edb at kemper.freedesktop.org
Fri Sep 16 20:10:01 UTC 2016
Module: Mesa
Branch: master
Commit: 1c8d4c694bfb95ab90f152da710a8a9459a20228
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=1c8d4c694bfb95ab90f152da710a8a9459a20228
Author: Serge Martin <edb+mesa at sigluy.net>
Date: Tue Aug 30 10:10:01 2016 +0200
clover: fix getting scalar args api size
This fix getting the size of a struct arg. vec3 types still work ok.
Only buit-in args need to have power of two alignment, getTypeAllocSize
reports the correct size in all cases.
Acked-by: Francisco Jerez <currojerez at riseup.net>
---
src/gallium/state_trackers/clover/llvm/codegen/common.cpp | 7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/src/gallium/state_trackers/clover/llvm/codegen/common.cpp b/src/gallium/state_trackers/clover/llvm/codegen/common.cpp
index 2d52fdc..834b06a 100644
--- a/src/gallium/state_trackers/clover/llvm/codegen/common.cpp
+++ b/src/gallium/state_trackers/clover/llvm/codegen/common.cpp
@@ -81,11 +81,10 @@ namespace {
// OpenCL 1.2 specification, Ch. 6.1.5: "A built-in data
// type that is not a power of two bytes in size must be
- // aligned to the next larger power of two". We need this
- // alignment for three element vectors, which have
- // non-power-of-2 store size.
+ // aligned to the next larger power of two.
+ // This rule applies to built-in types only, not structs or unions."
const unsigned arg_store_size = dl.getTypeStoreSize(arg_type);
- const unsigned arg_api_size = util_next_power_of_two(arg_store_size);
+ const unsigned arg_api_size = dl.getTypeAllocSize(arg_type);
const auto target_type = !arg_type->isIntegerTy() ? arg_type :
dl.getSmallestLegalIntType(mod.getContext(), arg_store_size * 8);
More information about the mesa-commit
mailing list