[Mesa-dev] [PATCH v2 7.5/18] intel/compiler: support negate and abs of half float immediates

Iago Toral itoral at igalia.com
Thu May 3 09:58:33 UTC 2018


On Thu, 2018-05-03 at 08:39 +0200, Iago Toral wrote:
> On Wed, 2018-05-02 at 17:57 -0700, Jason Ekstrand wrote:
> > Reviewed-by: Jason Ekstrand <jason at jlekstrand.net>
> > 
> > Have I reviewed everything?  Can we land shaderInt16 now?
> 
> Yes, all patches are reviewed now, thanks Jason.
> I'll send the final set of patches to Jenkins one last time and push
> them today  if we don't see any unexpected results.

I have just pushed the patches to master.
Iago
> Iago
> > --Jason
> > 
> > On Wed, May 2, 2018 at 5:18 PM, Jose Maria Casanova Crespo <jmcasan
> > ova at igalia.com> wrote:
> > > ---
> > > 
> > >  src/intel/compiler/brw_shader.cpp | 6 ++++--
> > > 
> > >  1 file changed, 4 insertions(+), 2 deletions(-)
> > > 
> > > 
> > > 
> > > diff --git a/src/intel/compiler/brw_shader.cpp
> > > b/src/intel/compiler/brw_shader.cpp
> > > 
> > > index 284c2e8233c..537defd05d9 100644
> > > 
> > > --- a/src/intel/compiler/brw_shader.cpp
> > > 
> > > +++ b/src/intel/compiler/brw_shader.cpp
> > > 
> > > @@ -605,7 +605,8 @@ brw_negate_immediate(enum brw_reg_type type,
> > > struct brw_reg *reg)
> > > 
> > >     case BRW_REGISTER_TYPE_V:
> > > 
> > >        assert(!"unimplemented: negate UV/V immediate");
> > > 
> > >     case BRW_REGISTER_TYPE_HF:
> > > 
> > > -      assert(!"unimplemented: negate HF immediate");
> > > 
> > > +      reg->ud ^= 0x80008000;
> > > 
> > > +      return true;
> > > 
> > >     case BRW_REGISTER_TYPE_NF:
> > > 
> > >        unreachable("no NF immediates");
> > > 
> > >     }
> > > 
> > > @@ -651,7 +652,8 @@ brw_abs_immediate(enum brw_reg_type type,
> > > struct brw_reg *reg)
> > > 
> > >     case BRW_REGISTER_TYPE_V:
> > > 
> > >        assert(!"unimplemented: abs V immediate");
> > > 
> > >     case BRW_REGISTER_TYPE_HF:
> > > 
> > > -      assert(!"unimplemented: abs HF immediate");
> > > 
> > > +      reg->ud &= ~0x80008000;
> > > 
> > > +      return true;
> > > 
> > >     case BRW_REGISTER_TYPE_NF:
> > > 
> > >        unreachable("no NF immediates");
> > > 
> > >     }
> > > 
> > > -- 
> > > 
> > > 2.14.3
> > > 
> > > 
> > > 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20180503/214187f9/attachment-0001.html>


More information about the mesa-dev mailing list