<html dir="ltr"><head></head><body style="text-align:left; direction:ltr;"><div>Sure, will do that.</div><div><br></div><div>On Fri, 2018-12-07 at 14:11 -0600, Jason Ekstrand wrote:</div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div dir="ltr">51 and 52 should probably be rolled together. For that matter, I don't think we need 6 patches just to add two SPIR-V capabilities and advertise one extension. Maybe roll the two SPIR-V patches together, add one or two for the extension and the enables?<br></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Dec 4, 2018 at 1:18 AM Iago Toral Quiroga <<a href="mailto:itoral@igalia.com">itoral@igalia.com</a>> wrote:<br></div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex">---<br>
src/intel/vulkan/anv_device.c | 2 +-<br>
1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/src/intel/vulkan/anv_device.c b/src/intel/vulkan/anv_device.c<br>
index caf25ad8a03..17b73c115cd 100644<br>
--- a/src/intel/vulkan/anv_device.c<br>
+++ b/src/intel/vulkan/anv_device.c<br>
@@ -971,7 +971,7 @@ void anv_GetPhysicalDeviceFeatures2(<br>
ANV_FROM_HANDLE(anv_physical_device, pdevice, physicalDevice);<br>
<br>
features->shaderFloat16 = pdevice->info.gen >= 8;<br>
- features->shaderInt8 = false;<br>
+ features->shaderInt8 = pdevice->info.gen >= 8;<br>
break;<br>
}<br>
<br>
</blockquote></div></blockquote></body></html>