[PATCH] D22108: AMDGPU: Only use legal inline immediates with kill pseudo

[Nicolai Hähnle via llvm-commits]

nhaehnle added a comment.

I agree, but the D3D designers in their infinite wisdom designed a KILL opcode with a floating point parameter, which we inherited via TGSI in Mesa, which in turn motivated this (indeed somewhat questionable) intrinsic.

Hmm. My concern is that (non-constant) KILL gets lowered to a floating point comparison, which treats NaNs specially. (i32)-1 is a NaN. I admit that it doesn't really matter since this patch uses a constant, but it still leaves me with an uncomfortable feeling.


http://reviews.llvm.org/D22108