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

2017-03-23 Thread Francisco Jerez
Samuel Iglesias Gonsálvez writes: > 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

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

2017-03-23 Thread Samuel Iglesias Gonsálvez
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

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

2017-03-22 Thread Francisco Jerez
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. Samuel Iglesias Gonsálvez writes: > Signed-off-by: Samuel Iglesias Gonsálvez

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

2017-03-20 Thread Samuel Iglesias Gonsálvez
Signed-off-by: Samuel Iglesias Gonsálvez --- src/intel/Makefile.sources | 2 +- src/intel/compiler/brw_fs.cpp | 2 +- src/intel/compiler/brw_fs.h