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

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 24 05:48:35 PDT 2022


foad added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUInstructionSelector.cpp:1297
+                     .addReg(Src1)
+                     .addImm(0); // omod
+  } else
----------------
Pierre-vh wrote:
> Does omod need to be selected too? How?
This operand is clamp, not omod, and for V_CMP it enables "signalling compare". But I'm not sure if we ever make use of it in the compiler.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136592



More information about the llvm-commits mailing list