[PATCH] D44521: [InstSimplify] fp_binop X, NaN --> NaN

Nicolai Hähnle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 17 05:41:08 PDT 2018


nhaehnle added a comment.

For the AMDGPU imm tests, you could please change the affected ones to operate on i32/i16 instead, so that the result uses v_add_u32? Thanks!

I don't expect this approach to work for the double cases, I think you can just leave those as-is.


https://reviews.llvm.org/D44521





More information about the llvm-commits mailing list