Or even better, cast it to uint32_t explicitly.
On Thu, May 30, 2019 at 8:23 PM Bas Nieuwenhuizen <b...@basnieuwenhuizen.nl> wrote: > > maybe use UINT32_MAX? > > otherwise r-b > > On Thu, May 30, 2019 at 6:50 PM Rhys Perry <pendingchao...@gmail.com> wrote: > > > > Fixes -Woverflow warnings with GCC 9.1.1 > > > > Signed-off-by: Rhys Perry <pendingchao...@gmail.com> > > --- > > src/amd/vulkan/si_cmd_buffer.c | 8 ++++---- > > 1 file changed, 4 insertions(+), 4 deletions(-) > > > > diff --git a/src/amd/vulkan/si_cmd_buffer.c b/src/amd/vulkan/si_cmd_buffer.c > > index aae8d578c10..d87c00b94e9 100644 > > --- a/src/amd/vulkan/si_cmd_buffer.c > > +++ b/src/amd/vulkan/si_cmd_buffer.c > > @@ -1360,7 +1360,7 @@ void radv_emit_default_sample_locations(struct > > radeon_cmdbuf *cs, int nr_samples > > default: > > case 1: > > radeon_set_context_reg_seq(cs, > > R_028BD4_PA_SC_CENTROID_PRIORITY_0, 2); > > - radeon_emit(cs, centroid_priority_1x); > > + radeon_emit(cs, centroid_priority_1x & 0xffffffff); > > radeon_emit(cs, centroid_priority_1x >> 32); > > radeon_set_context_reg(cs, > > R_028BF8_PA_SC_AA_SAMPLE_LOCS_PIXEL_X0Y0_0, sample_locs_1x); > > radeon_set_context_reg(cs, > > R_028C08_PA_SC_AA_SAMPLE_LOCS_PIXEL_X1Y0_0, sample_locs_1x); > > @@ -1369,7 +1369,7 @@ void radv_emit_default_sample_locations(struct > > radeon_cmdbuf *cs, int nr_samples > > break; > > case 2: > > radeon_set_context_reg_seq(cs, > > R_028BD4_PA_SC_CENTROID_PRIORITY_0, 2); > > - radeon_emit(cs, centroid_priority_2x); > > + radeon_emit(cs, centroid_priority_2x & 0xffffffff); > > radeon_emit(cs, centroid_priority_2x >> 32); > > radeon_set_context_reg(cs, > > R_028BF8_PA_SC_AA_SAMPLE_LOCS_PIXEL_X0Y0_0, sample_locs_2x); > > radeon_set_context_reg(cs, > > R_028C08_PA_SC_AA_SAMPLE_LOCS_PIXEL_X1Y0_0, sample_locs_2x); > > @@ -1378,7 +1378,7 @@ void radv_emit_default_sample_locations(struct > > radeon_cmdbuf *cs, int nr_samples > > break; > > case 4: > > radeon_set_context_reg_seq(cs, > > R_028BD4_PA_SC_CENTROID_PRIORITY_0, 2); > > - radeon_emit(cs, centroid_priority_4x); > > + radeon_emit(cs, centroid_priority_4x & 0xffffffff); > > radeon_emit(cs, centroid_priority_4x >> 32); > > radeon_set_context_reg(cs, > > R_028BF8_PA_SC_AA_SAMPLE_LOCS_PIXEL_X0Y0_0, sample_locs_4x); > > radeon_set_context_reg(cs, > > R_028C08_PA_SC_AA_SAMPLE_LOCS_PIXEL_X1Y0_0, sample_locs_4x); > > @@ -1387,7 +1387,7 @@ void radv_emit_default_sample_locations(struct > > radeon_cmdbuf *cs, int nr_samples > > break; > > case 8: > > radeon_set_context_reg_seq(cs, > > R_028BD4_PA_SC_CENTROID_PRIORITY_0, 2); > > - radeon_emit(cs, centroid_priority_8x); > > + radeon_emit(cs, centroid_priority_8x & 0xffffffff); > > radeon_emit(cs, centroid_priority_8x >> 32); > > radeon_set_context_reg_seq(cs, > > R_028BF8_PA_SC_AA_SAMPLE_LOCS_PIXEL_X0Y0_0, 14); > > radeon_emit_array(cs, sample_locs_8x, 4); > > -- > > 2.21.0 > > > > _______________________________________________ > > mesa-dev mailing list > > mesa-dev@lists.freedesktop.org > > https://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev