<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Mar 29, 2017 at 12:41 AM, Eduardo Lima Mitev <span dir="ltr"><<a href="mailto:elima@igalia.com" target="_blank">elima@igalia.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Both patches need rebase, but look fine otherwise.<br></blockquote><div><br></div><div>The first has already landed (I think).  The second definitely needs rebasing.  Yesterday, I rebased it on top of the other two constant_expressions fixup patches I sent out: <a href="https://patchwork.freedesktop.org/series/21244/">https://patchwork.freedesktop.org/series/21244/</a>  It would be nice if that series landed first as it cleans things up substantially.<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">
Series is:<br>
<br>
Reviewed-by: Eduardo Lima Mitev <<a href="mailto:elima@igalia.com">elima@igalia.com</a>><br><div class="gmail-HOEnZb"><div class="gmail-h5"></div></div></blockquote><div><br></div><div>Thanks!<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"><div class="gmail-HOEnZb"><div class="gmail-h5">
On 03/09/2017 11:05 PM, Jason Ekstrand wrote:<br>
> This tiny series adds support in NIR for 8 and 16-bit types.  In<br>
> particular, it now supports int8_t, uint8_t, int16_t, uint16_t, and<br>
> float16_t.  No 8-bit floating-point type is supported because 8-bit float<br>
> would be stupid.<br>
><br>
> These patches have been tested in Jenkins but no 8 or 16-bit code has been<br>
> run through it yet.  Even if we're people don't want to land the second<br>
> patch (due to not having a vertical slice), I'd like to land the first<br>
> refactor patch.<br>
><br>
> Jason Ekstrand (2):<br>
>   nir/constant_expressions: Refactor helper functions<br>
>   nir: Add support for 8 and 16-bit types<br>
><br>
>  src/compiler/nir/nir.h                       |  4 ++<br>
>  src/compiler/nir/nir_constant_<wbr>expressions.py | 67 +++++++++++++++++-----------<br>
>  src/compiler/nir/nir_opcodes.<wbr>py              |  6 ++-<br>
>  3 files changed, 51 insertions(+), 26 deletions(-)<br>
><br>
<br>
</div></div></blockquote></div><br></div></div>