[PATCH] D11829: AMDGPU/SI: Re-define AMDGPUISD:CLAMP as always clamping between 0.0. and 1.0

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

In https://reviews.llvm.org/D11829#681483, @artem.tamazov wrote:

> In https://reviews.llvm.org/D11829#681459, @artem.tamazov wrote:
>
> > Starting from Gfx9, min/max/med3 do support denorm modes on data path.
>
>
> Actually, I do not see anything wrong wrt usage of min/max/med3 for now.
>  Min/max/med3 is even better than ADD because "ADD X, 0.0"does not preserve the sign of X when X==-0.0.


@artem.tamazov Do you know the answer to the denormal questions I added in comments? Most importantly are denormals really not supported with clamp? And if not for f32, are they still not supported for f16/f64? Does clamp work with f64 instructions?


https://reviews.llvm.org/D11829