<html dir="ltr"><head></head><body style="text-align:left; direction:ltr;"><div>Sorry about that, I split them like this because one change is for NIR and the other one is for Intel so I thought it was preferred to keep changes like this in separate patches. Anyway, I understand that for small changes like this it is not helpful, I'll keep that in mind for future patches.</div><div><br></div><div>On Thu, 2019-01-17 at 14:04 -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">Again, please squash with the previous patch. Splitting stuff this granular just makes review harder.<br></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Jan 15, 2019 at 7:54 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">Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net" target="_blank">jason@jlekstrand.net</a>><br>
---<br>
src/intel/compiler/brw_compiler.c | 1 +<br>
1 file changed, 1 insertion(+)<br>
<br>
diff --git a/src/intel/compiler/brw_compiler.c b/src/intel/compiler/brw_compiler.c<br>
index f885e79c3e6..04a1a7cac4e 100644<br>
--- a/src/intel/compiler/brw_compiler.c<br>
+++ b/src/intel/compiler/brw_compiler.c<br>
@@ -33,6 +33,7 @@<br>
.lower_sub = true, \<br>
.lower_fdiv = true, \<br>
.lower_scmp = true, \<br>
+ .lower_flrp16 = true, \<br>
.lower_fmod16 = true, \<br>
.lower_fmod32 = true, \<br>
.lower_fmod64 = false, \<br>
</blockquote></div></blockquote></body></html>