[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