[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
More information about the llvm-commits