<div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Mar 14, 2019 at 7:08 PM Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Eleni,<br>
<br>
I made a comment on one of your patches and then I started getting lost in<br>
the churn.  It's easier to see what's going on if things are reordered and<br>
refactored a bit.  Unfortunately, what needed to be done was hard to<br>
explain and it was easier for me to just do the code motion and re-send the<br>
series.  I hope you understand.  This is one of those cases where it's<br>
easier for me to write code than English.<br>
<br>
One of the results of this refactor is that gen8 and gen7 changes happen at<br>
the same time.<br>
<br>
--Jason<br>
<br>
<br>
<br>
Eleni Maria Stea (9):<br>
  anv: Added the VK_EXT_sample_locations extension to the anv_extensions<br>
    list<br>
  anv: Set the values for the<br>
    VkPhysicalDeviceSampleLocationsPropertiesEXT<br>
  anv: Implemented the vkGetPhysicalDeviceMultisamplePropertiesEXT<br>
  anv/state: Take explicit sample locations in emit helpers<br>
  anv: Add support for non-dynamic sample locations<br></blockquote><div><br></div><div>The above patches are all</div><div><br></div><div>Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
  anv: Added support for dynamic sample locations<br>
  anv: Optimized the emission of the default locations on Gen8+<br></blockquote><div><br></div><div>I've got some concerns about these two.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
  anv: Removed unused header file<br>
  anv: Enabled the VK_EXT_sample_locations extension<br></blockquote><div><br></div><div>These two are also</div><div><br></div><div>Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Jason Ekstrand (2):<br>
  anv/pipeline: Refactor 3DSTATE_SAMPLE_MASK setup<br>
  anv: Break SAMPLE_PATTERN and MULTISAMPLE emit into helpers<br>
<br>
 src/intel/common/gen_sample_positions.h |  57 +++++++++<br>
 src/intel/vulkan/anv_cmd_buffer.c       |  17 +++<br>
 src/intel/vulkan/anv_device.c           |  45 ++++++++<br>
 src/intel/vulkan/anv_extensions.py      |   1 +<br>
 src/intel/vulkan/anv_genX.h             |   6 +<br>
 src/intel/vulkan/anv_private.h          |   7 ++<br>
 src/intel/vulkan/genX_blorp_exec.c      |   1 -<br>
 src/intel/vulkan/genX_cmd_buffer.c      |  12 ++<br>
 src/intel/vulkan/genX_pipeline.c        |  93 +++++++--------<br>
 src/intel/vulkan/genX_state.c           | 147 ++++++++++++++++++++----<br>
 10 files changed, 315 insertions(+), 71 deletions(-)<br>
<br>
-- <br>
2.20.1<br>
<br>
</blockquote></div></div>