[llvm] [SelectionDAG][RISCV] Operations with static rounding (PR #100999)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 1 04:07:34 PDT 2024


================
@@ -8131,15 +8131,69 @@ void SelectionDAGBuilder::visitIntrinsicCall(const CallInst &I,
 void SelectionDAGBuilder::visitConstrainedFPIntrinsic(
     const ConstrainedFPIntrinsic &FPI) {
   SDLoc sdl = getCurSDLoc();
+  const TargetLowering &TLI = DAG.getTargetLoweringInfo();
+  EVT VT = TLI.getValueType(DAG.getDataLayout(), FPI.getType());
+  fp::ExceptionBehavior EB = *FPI.getExceptionBehavior();
+  std::optional<RoundingMode> RM = FPI.getRoundingMode();
+
+  SDNodeFlags Flags;
+  if (EB == fp::ExceptionBehavior::ebIgnore)
+    Flags.setNoFPExcept(true);
+
+  if (auto *FPOp = dyn_cast<FPMathOperator>(&FPI))
+    Flags.copyFMF(*FPOp);
+
+  bool UseStaticRounding = EB == fp::ExceptionBehavior::ebIgnore && RM &&
+                           *RM != RoundingMode::Dynamic &&
+                           TLI.isStaticRoundingSupportedFor(FPI);
+  unsigned Opcode = 0;
+  SmallVector<SDValue, 4> Opers;
+  for (unsigned I = 0, E = FPI.getNonMetadataArgCount(); I != E; ++I)
+    Opers.push_back(getValue(FPI.getArgOperand(I)));
+
+  if (UseStaticRounding) {
+    switch (FPI.getIntrinsicID()) {
----------------
arsenm wrote:

It still doesn't make sense to me to select the constrained intrinsics to these operations. The metadata indicates that the runtime mode is this already, there isn't a benefit to using an instruction with a fixed mode 

https://github.com/llvm/llvm-project/pull/100999


More information about the llvm-commits mailing list