[PATCH] D136592: [AMDGPU][GISel] Select llvm.amdgcn.fcmp intrinsics

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUInstructionSelector.cpp:1167
+                  AMDGPU::V_CMP_NEQ_F32_e64, AMDGPU::V_CMP_NEQ_F64_e64);
+    return -1;
   }
----------------
I think the TRUE/FALSE cases should trivially select through, no fold to constant. Although these intrinsics are deprecated


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136592