[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 08:50:25 PDT 2016


arsenm added a comment.

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

> I don't understand why changing the type to i32 is necessary. Isn't -1.0 also a legal inline immediate practically everywhere?


It's not, it's just more natural to express a boolean with an integer


http://reviews.llvm.org/D22108





More information about the llvm-commits mailing list