[Mesa-dev] [PATCH v4 08/28] i965/fs: rename lower_d2x to lower_narrow_conversions

Samuel Iglesias Gonsálvez siglesias at igalia.com
Thu Mar 23 08:00:26 UTC 2017


On Wed, 2017-03-22 at 15:49 -0700, Francisco Jerez wrote:
> I'd rename this to lower_conversions instead since after your
> previous
> patch it's going to take care of handling unsupported conversions
> which
> aren't necessarily to a narrower type.
> 

OK. Does this patch get your R-b then?

Sam

> Samuel Iglesias Gonsálvez <siglesias at igalia.com> writes:
> 
> > Signed-off-by: Samuel Iglesias Gonsálvez <siglesias at igalia.com>
> > ---
> >  src/intel/Makefile.sources                                        
> >       | 2 +-
> >  src/intel/compiler/brw_fs.cpp                                     
> >       | 2 +-
> >  src/intel/compiler/brw_fs.h                                       
> >       | 2 +-
> >  .../{brw_fs_lower_d2x.cpp =>
> > brw_fs_lower_narrow_conversions.cpp}       | 2 +-
> >  4 files changed, 4 insertions(+), 4 deletions(-)
> >  rename src/intel/compiler/{brw_fs_lower_d2x.cpp =>
> > brw_fs_lower_narrow_conversions.cpp} (99%)
> > 
> > diff --git a/src/intel/Makefile.sources
> > b/src/intel/Makefile.sources
> > index 4eaf380492f..a9810887f80 100644
> > --- a/src/intel/Makefile.sources
> > +++ b/src/intel/Makefile.sources
> > @@ -42,7 +42,7 @@ COMPILER_FILES = \
> >  	compiler/brw_fs.h \
> >  	compiler/brw_fs_live_variables.cpp \
> >  	compiler/brw_fs_live_variables.h \
> > -	compiler/brw_fs_lower_d2x.cpp \
> > +	compiler/brw_fs_lower_narrow_conversions.cpp \
> >  	compiler/brw_fs_lower_pack.cpp \
> >  	compiler/brw_fs_nir.cpp \
> >  	compiler/brw_fs_reg_allocate.cpp \
> > diff --git a/src/intel/compiler/brw_fs.cpp
> > b/src/intel/compiler/brw_fs.cpp
> > index 8612a83805d..61ac981842e 100644
> > --- a/src/intel/compiler/brw_fs.cpp
> > +++ b/src/intel/compiler/brw_fs.cpp
> > @@ -5740,7 +5740,7 @@ fs_visitor::optimize()
> >        OPT(dead_code_eliminate);
> >     }
> >  
> > -   if (OPT(lower_d2x)) {
> > +   if (OPT(lower_narrow_conversions)) {
> >        OPT(opt_copy_propagation);
> >        OPT(dead_code_eliminate);
> >        OPT(lower_simd_width);
> > diff --git a/src/intel/compiler/brw_fs.h
> > b/src/intel/compiler/brw_fs.h
> > index f3d36848e54..416397fabcd 100644
> > --- a/src/intel/compiler/brw_fs.h
> > +++ b/src/intel/compiler/brw_fs.h
> > @@ -160,7 +160,7 @@ public:
> >     void lower_uniform_pull_constant_loads();
> >     bool lower_load_payload();
> >     bool lower_pack();
> > -   bool lower_d2x();
> > +   bool lower_narrow_conversions();
> >     bool lower_logical_sends();
> >     bool lower_integer_multiplication();
> >     bool lower_minmax();
> > diff --git a/src/intel/compiler/brw_fs_lower_d2x.cpp
> > b/src/intel/compiler/brw_fs_lower_narrow_conversions.cpp
> > similarity index 99%
> > rename from src/intel/compiler/brw_fs_lower_d2x.cpp
> > rename to src/intel/compiler/brw_fs_lower_narrow_conversions.cpp
> > index fa25d313dcd..02a60b07eec 100644
> > --- a/src/intel/compiler/brw_fs_lower_d2x.cpp
> > +++ b/src/intel/compiler/brw_fs_lower_narrow_conversions.cpp
> > @@ -44,7 +44,7 @@ supports_type_conversion(fs_inst *inst) {
> >  }
> >  
> >  bool
> > -fs_visitor::lower_d2x()
> > +fs_visitor::lower_narrow_conversions()
> >  {
> >     bool progress = false;
> >  
> > -- 
> > 2.11.0
> > 
> > _______________________________________________
> > mesa-dev mailing list
> > mesa-dev at lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/mesa-dev
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170323/ff08457d/attachment-0001.sig>


More information about the mesa-dev mailing list