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

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 8 10:44:40 PDT 2016


arsenm added a comment.

In http://reviews.llvm.org/D22108#478244, @nhaehnle wrote:

> 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.


The intrinsic itself's type doesn't change, this is just an internal implementation detail


http://reviews.llvm.org/D22108





More information about the llvm-commits mailing list