[PATCH] D143420: AMDGPU: Replace certain llvm.amdgcn.class uses with llvm.is.fpclass

[Jay Foad via Phabricator via llvm-commits]

foad added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUInstCombineIntrinsic.cpp:438
+    if (isa<UndefValue>(Src0))
+      return IC.replaceInstUsesWith(II, UndefValue::get(II.getType()));
 
----------------
I'm not convinced this is sound. E.g. if Src1 is dynamically zero (resp. all-ones) then the result should be false (resp. true) even if Src0 is undef. But that's a pre-existing problem.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D143420/new/

https://reviews.llvm.org/D143420