<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Apr 23, 2018 at 1:12 PM, Samuel Pitoiset <span dir="ltr"><<a href="mailto:samuel.pitoiset@gmail.com" target="_blank">samuel.pitoiset@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br>
<br>
On 04/23/2018 06:55 PM, Nicolai Hähnle wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 23.04.2018 17:52, Samuel Pitoiset wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
This fixes crashes for the following CTS:<br>
dEQP-VK.glsl.texture_functions<wbr>.query.texturequerylod.*<br>
<br>
Fixes: 625dcbbc456 ("amd/common: pass address components individually to<br>
ac_build_image_intrinsic")<br>
Cc: 18.1 <<a href="mailto:mesa-stable@lists.freedesktop.org" target="_blank">mesa-stable@lists.freedesktop<wbr>.org</a>><br>
Signed-off-by: Samuel Pitoiset <<a href="mailto:samuel.pitoiset@gmail.com" target="_blank">samuel.pitoiset@gmail.com</a>><br>
---<br>
  src/amd/common/ac_llvm_build.c | 13 +++++++++++++<br>
  1 file changed, 13 insertions(+)<br>
<br>
diff --git a/src/amd/common/ac_llvm_build<wbr>.c b/src/amd/common/ac_llvm_build<wbr>.c<br>
index 02739f9da9c..d5bad3eeea3 100644<br>
--- a/src/amd/common/ac_llvm_build<wbr>.c<br>
+++ b/src/amd/common/ac_llvm_build<wbr>.c<br>
@@ -1521,6 +1521,19 @@ LLVMValueRef ac_build_image_opcode(struct ac_llvm_context *ctx,<br>
      LLVMValueRef addr;<br>
      unsigned num_addr = 0;<br>
+    if (a->opcode == ac_image_get_lod) {<br>
+        switch (a->dim) {<br>
+        case ac_image_1darray:<br>
+            num_coords = 1;<br>
+            break;<br>
+        case ac_image_2darray:<br>
+            num_coords = 2;<br>
</blockquote>
<br>
This is probably also needed for cube maps, isn't it?<br>
</blockquote>
<br></div></div>
cubes should have 3 coordinates for textureQueryLod(), no?<span class="im HOEnZb"><br></span></blockquote><div><br></div><div>Cubemaps are the same as 2D arrays.<br><br></div><div>Marek<br></div></div><br></div></div>