On Mar 23, 2016 2:59 AM, "Samuel Iglesias Gonsálvez" <<a href="mailto:siglesias@igalia.com">siglesias@igalia.com</a>> wrote: > > > > On 22/03/16 16:17, Samuel Iglesias Gonsálvez wrote: > > On 21/03/16 23:49, Jason Ekstrand wrote: > >> On Mon, Mar 21, 2016 at 5:05 AM, Samuel Iglesias Gonsálvez < > >> <a href="mailto:siglesias@igalia.com">siglesias@igalia.com</a>> wrote: > >> > >>> From: Connor Abbott <<a href="mailto:connor.w.abbott@intel.com">connor.w.abbott@intel.com</a>> > >>> > >>> --- > >>> src/compiler/nir/nir_print.c | 5 ++++- > >>> 1 file changed, 4 insertions(+), 1 deletion(-) > >>> > >>> diff --git a/src/compiler/nir/nir_print.c b/src/compiler/nir/nir_print.c > >>> index 30a8233..0b3f954 100644 > >>> --- a/src/compiler/nir/nir_print.c > >>> +++ b/src/compiler/nir/nir_print.c > >>> @@ -719,7 +719,10 @@ print_load_const_instr(nir_load_const_instr *instr, > >>> print_state *state) > >>> * and then print the float in a comment for readability. > >>> */ > >>> > >>> - fprintf(fp, "0x%08x /* %f */", instr->value.u32[i], > >>> instr->value.f32[i]); > >>> + if (instr->def.bit_size == 64) > >>> + fprintf(fp, "%f", instr->value.f64[i]); > >>> > >> > >> Let's print out the 64-bit integer here as well. 64-bit integer support > >> may happen. > >> > > > > OK, I will do the change. > > > > Sam > > > > Looking at this patch and the other two nir/print patches, it makes > sense to squash them altogether in one patch. Is it OK for you? Yup. Sounds good > Sam > > >> > >>> + else > >>> + fprintf(fp, "0x%08x /* %f */", instr->value.u32[i], > >>> instr->value.f32[i]); > >>> } > >>> > >>> fprintf(fp, ")"); > >>> -- > >>> 2.5.0 > >>> > >>> _______________________________________________ > >>> mesa-dev mailing list > >>> <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a> > >>> <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a> > >>> > >>