On Mon, Feb 22, 2016 at 9:40 PM, Francisco Jerez <curroje...@riseup.net> wrote:
> So the result is of float type if we're implementing the float
> overload of imageAtomicExchange. This is the only back-end change
> required to support OES_shader_image_atomic AFAICT.
> ---
> src/mesa/drivers/dri/i965/brw_fs_surface_builder.cpp | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_fs_surface_builder.cpp
> b/src/mesa/drivers/dri/i965/brw_fs_surface_builder.cpp
> index 081dbad..75734d2 100644
> --- a/src/mesa/drivers/dri/i965/brw_fs_surface_builder.cpp
> +++ b/src/mesa/drivers/dri/i965/brw_fs_surface_builder.cpp
> @@ -1125,7 +1125,7 @@ namespace brw {
> if (rsize && pred)
> set_predicate(pred, bld.SEL(tmp, tmp, brw_imm_d(0)));
>
> - return tmp;
> + return retype(tmp, src0.type);
I had chosen to stick this into emit_typed_atomic -- the typed-ness
seemed to fit better there. Your call though, this is just a thought.
> }
> }
> }
> --
> 2.7.0
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
