<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<tt>Hi, Thanks!<br>
<br>
I finally got CI results with ALL relevant tests and my
implementation works on everything but Haswell.<br>
So I'll fix whatever is wrong there (hope that's just something
small) and send the final version.<br>
<br>
There are still some tests for VK-GL-CTS which are not merged in
master e.g.<br>
<a class="moz-txt-link-freetext" href="https://github.com/KhronosGroup/VK-GL-CTS/pull/136">https://github.com/KhronosGroup/VK-GL-CTS/pull/136</a> and there are
some unpublished tests for VK_EXT_conditional_rendering from other
devs<br>
which I have no idea when will be merged</tt><tt><tt>. I don't
think this will prevent merging my patches but still I would be
more confident if CI have ALL tests.<br>
<br>
And yes, I also was </tt></tt><tt><tt><span
class="gt-baf-term-text"><span class="gt-baf-cell
gt-baf-word-clickable">quite slow with my patches.<br>
</span></span></tt></tt>
<div class="moz-cite-prefix"><br>
On 1/8/19 11:13 AM, Erik Faye-Lund wrote:<br>
</div>
<blockquote type="cite"
cite="mid:9cad2a5d17f56c83e1d615856a77442f3d6a44df.camel@collabora.com">
<pre class="moz-quote-pre" wrap="">On Wed, 2018-10-17 at 14:58 +0300, Danylo Piliaiev wrote:
</pre>
<blockquote type="cite">
<pre class="moz-quote-pre" wrap="">This series implement VK_KHR_draw_indirect_count and
VK_EXT_conditional_rendering extensions.
They are implemented together because they are highly interweaved.
There are already tests in VK_CTS for VK_KHR_draw_indirect_count and
I made a pull request with
the tests for VK_EXT_conditional_rendering (
<a class="moz-txt-link-freetext" href="https://github.com/KhronosGroup/VK-GL-CTS/pull/131">https://github.com/KhronosGroup/VK-GL-CTS/pull/131</a>).
VK_KHR_draw_indirect_count is implemented for gen7+.
VK_EXT_conditional_rendering is implemented for gen7.5+ because it
requires MI_MATH to be
implemented correctly.
Since part of the tests aren't in VK-GL-CTS master I'm not sure how
to test the implementation
of VK_EXT_conditional_rendering with my tests on CI. Could anyone
help me with this?
Also the one thing I'm uncertain of is described in the last patch.
Many thanks to Jason Ekstrand for the help with the extensions.
</pre>
</blockquote>
<pre class="moz-quote-pre" wrap="">
Hey, great work :)
What's holding this series back? I'm very interested in this
functionality due to my work on Zink... :)
</pre>
</blockquote>
<br>
</body>
</html>