Quoting Ian Romanick (2019-01-02 12:57:26) > On 12/19/18 9:25 AM, Dylan Baker wrote: > > Quoting Jonathan Marek (2018-12-19 08:39:53) > >> Mainly a copy of nir_lower_bool_to_int32, but with float opcodes. > >> > >> Signed-off-by: Jonathan Marek <jonat...@marek.ca> > >> --- > >> src/compiler/Makefile.sources | 1 + > >> src/compiler/nir/meson.build | 3 +- > >> src/compiler/nir/nir.h | 1 + > >> src/compiler/nir/nir_lower_bool_to_float.c | 165 +++++++++++++++++++++ > >> 4 files changed, 169 insertions(+), 1 deletion(-) > >> create mode 100644 src/compiler/nir/nir_lower_bool_to_float.c > >> > >> diff --git a/src/compiler/Makefile.sources b/src/compiler/Makefile.sources > >> index ef47bdb33b..39eaedc658 100644 > >> --- a/src/compiler/Makefile.sources > >> +++ b/src/compiler/Makefile.sources > >> @@ -231,6 +231,7 @@ NIR_FILES = \ > >> nir/nir_lower_atomics_to_ssbo.c \ > >> nir/nir_lower_bitmap.c \ > >> nir/nir_lower_bit_size.c \ > >> + nir/nir_lower_bool_to_float.c \ > >> nir/nir_lower_bool_to_int32.c \ > >> nir/nir_lower_clamp_color_outputs.c \ > >> nir/nir_lower_clip.c \ > >> diff --git a/src/compiler/nir/meson.build b/src/compiler/nir/meson.build > >> index e252f64539..f1016104af 100644 > >> --- a/src/compiler/nir/meson.build > >> +++ b/src/compiler/nir/meson.build > >> @@ -114,6 +114,7 @@ files_libnir = files( > >> 'nir_lower_alpha_test.c', > >> 'nir_lower_atomics_to_ssbo.c', > >> 'nir_lower_bitmap.c', > >> + 'nir_lower_bool_to_float.c', > >> 'nir_lower_bool_to_int32.c', > >> 'nir_lower_clamp_color_outputs.c', > >> 'nir_lower_clip.c', > >> @@ -248,7 +249,7 @@ if with_tests > >> include_directories : [inc_common], > >> dependencies : [dep_thread, idep_gtest, idep_nir], > >> link_with : libmesa_util, > >> - ), > >> + ), > > > > This looks like stray whitespace? > > It's deleting a stray (incorrect?) whitespace. I'm usually not fond of > slipping unrelated changes into a commit... but who's going to send a > 1-line patch that deletes a single space character? :)
Ah, it wasn't clear whether it was removing whitespace or adding it :)
signature.asc
Description: signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev