<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Nov 6, 2017 at 7:26 PM, Jason Ekstrand <span dir="ltr"><<a href="mailto:jason@jlekstrand.net" target="_blank">jason@jlekstrand.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div class="h5"><div class="gmail_extra"><div class="gmail_quote">On Mon, Nov 6, 2017 at 6:33 PM, Chad Versace <span dir="ltr"><<a href="mailto:chadversary@chromium.org" target="_blank">chadversary@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Mon 06 Nov 2017, Jason Ekstrand wrote:<br>
> On Mon, Nov 6, 2017 at 12:18 PM, Chad Versace <[1]<a href="mailto:chadversary@chromium.org" target="_blank">chadversary@chromium.org</a>><br>
<span>> wrote:<br>
><br>
>     Jason, I tested this series against the khronos-internal vk-gl-cts and<br>
>     found an assertion failure in src/compiler/spirv. Any thoughts?<br>
><br>
>     I haven't debugged yet because I don't grok these parts of Mesa.<br>
><br>
>     vk-gl-cts<br>
><br>
>         commit a24448cdd72ffdbd8f7f571886625b<wbr>8a53100979<br>
><br>
>     mesa<br>
><br>
>         refs/tags/chadv/test/anv-varia<wbr>ble-pointers-2017-11-06-r1<br>
</span>>         cgit: [2]<a href="http://git.kiwitree.net/cgit/~chadv/mesa/tag/?h=chadv/test/" rel="noreferrer" target="_blank">http://git.kiwitree.net/cgi<wbr>t/~chadv/mesa/tag/?h=chadv/tes<wbr>t/</a><br>
<span>>     anv-variable-pointers-2017-<wbr>11-06-r1<br>
>         base: master 4bcb48b "radv: add initial copy descriptor support. (v2)"<br>
><br>
>     error<br>
><br>
>         Test case 'dEQP-VK.spirv_assembly.instru<wbr>ction.compute.variable_<br>
>     pointers.complex_types_comput<wbr>e.opptraccesschain_<br>
>     matrices_two_buffers_second_i<wbr>nput'..<br>
>         INTEL-MESA: debug: anv_GetPhysicalDeviceFeatures2<wbr>KHR: ignored<br>
>     VkStructureType VK_STRUCTURE_TYPE_PHYSICAL_DEV<wbr>ICE_16BIT_STORAGE_FEATURES_<br>
>     KHR(1000083000)<br>
>         deqp-vk: ../../../../../src/mesa/src/co<wbr>mpiler/spirv/vtn_variables.<br>
>     c:174: vtn_ssa_offset_pointer_derefer<wbr>ence: Assertion `offset' failed.<br>
><br>
><br>
> That looks very much like a test bug that I fixed.  The CLs have been merged<br>
> into vulkan-gl-cts-1.0.2 but I don't know if that's been merged forward yet. <br>
> Try that CTS version and see if it still fails.  It's CLs #1864 and #1863 in<br>
> gerrit if you're interested.<br>
<br>
</span>Weird. I'm testing with vk-gl-cts master a24448c, which contains both of<br>
those CLs.<br>
<br>
To be safe, I did a clean build of libdrm, mesa, and vk-gl-cts (with<br>
external/fetch_sources.py, of course). The failure remained.<br>
<br>
Maybe it's a toolchain problem?<br>
</blockquote></div></div><div class="gmail_extra"><br></div></div></div><div class="gmail_extra">No, it's because I failed at finding 100% of the broken tests.  New CL incoming.<br></div></div>
</blockquote></div></div><div class="gmail_extra"><br></div><div class="gmail_extra">The tests are fixed in CL #1915.  I feel like a dork now...<br></div></div>