<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - Struct / union sizes being calculated incorrectly"
href="https://bugs.freedesktop.org/show_bug.cgi?id=91556#c7">Comment # 7</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - Struct / union sizes being calculated incorrectly"
href="https://bugs.freedesktop.org/show_bug.cgi?id=91556">bug 91556</a>
from <span class="vcard"><a class="email" href="mailto:edb+freedesktop@sigluy.net" title="Serge Martin <edb+freedesktop@sigluy.net>"> <span class="fn">Serge Martin</span></a>
</span></b>
<pre>This simple change should fix the size calculation.
However the kernel still receive garbage, but we are working on it since
GROMACS have the same problem.
Also the serie found at
<a href="https://lists.freedesktop.org/archives/piglit/2016-February/019074.html">https://lists.freedesktop.org/archives/piglit/2016-February/019074.html</a>
is tracking this error
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b/src/gallium/state_trackers/clover/llvm/invocation.cpp
index 4d11c24..74c9511 100644
--- a/src/gallium/state_trackers/clover/llvm/invocation.cpp
+++ b/src/gallium/state_trackers/clover/llvm/invocation.cpp
@@ -547,8 +547,10 @@ namespace {
module::argument::sign_ext :
module::argument::zero_ext);
+ const unsigned arg_store_alloc = TD.getTypeAllocSize(arg_type);
+
args.push_back(
- module::argument(module::argument::scalar, arg_api_size,
+ module::argument(module::argument::scalar, arg_store_alloc,
target_size, target_align, ext_type));
}
}</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
<li>You are the QA Contact for the bug.</li>
</ul>
</body>
</html>