diff --git a/llvm/docs/LangRef.rst b/llvm/docs/LangRef.rst --- a/llvm/docs/LangRef.rst +++ b/llvm/docs/LangRef.rst @@ -19188,6 +19188,79 @@ Other values may be used to represent additional rounding modes, supported by a target. These values are target-specific. +'``llvm.get.fpenv``' Intrinsic +^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ + +Syntax: +""""""" + +:: + + declare @llvm.get.fpenv() + +Overview: +""""""""" + +The '``llvm.get.fpenv``' intrinsic returns floating point environment. The return +value type is platform-specific. + +Semantics: +"""""""""" + +The '``llvm.get.fpenv``' intrinsic reads the current floating point environment +and returns it as integer value. + +'``llvm.set.fpenv``' Intrinsic +^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ + +Syntax: +""""""" + +:: + + declare void @llvm.set.fpenv( ) + +Overview: +""""""""" + +The '``llvm.set.fpenv``' intrinsic sets floating point environment. + +Arguments: +"""""""""" + +The argument is an integer representing the floating point environment. The type +of the integer is platform-specific. + +Semantics: +"""""""""" + +The '``llvm.set.fpenv``' intrinsic sets the current floating point environment +to the state specified by the argument. The state may be previously obtained by a +call to '``llvm.get.fpenv``' or synthesised in platform-dependent way. + +'``llvm.reset.fpenv``' Intrinsic +^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ + +Syntax: +""""""" + +:: + + declare void @llvm.reset.fpenv() + +Overview: +""""""""" + +The '``llvm.reset.fpenv``' intrinsic sets default floating point environment. + +Semantics: +"""""""""" + +The '``llvm.reset.fpenv``' intrinsic sets the current floating point environment +to default state. It is similar to the call 'fesetenv(FE_DFL_ENV)', however this +function does not return any value. + + General Intrinsics ------------------ diff --git a/llvm/include/llvm/CodeGen/ISDOpcodes.h b/llvm/include/llvm/CodeGen/ISDOpcodes.h --- a/llvm/include/llvm/CodeGen/ISDOpcodes.h +++ b/llvm/include/llvm/CodeGen/ISDOpcodes.h @@ -822,6 +822,18 @@ /// FSINCOS - Compute both fsin and fcos as a single operation. FSINCOS, + /// Gets the current floating point environment. The first operand is token + /// chain. + GET_FPENV, + + /// Sets the current floating point environment. The first operand is token + /// chain, the second is FP environment, represented by integer value. + SET_FPENV, + + /// Set floating point environment to default state. The first operand is + /// token chain. + RESET_FPENV, + /// LOAD and STORE have token chains as their first operand, then the same /// operands as an LLVM load/store instruction, then an offset node that /// is added / subtracted from the base pointer to form the address (for diff --git a/llvm/include/llvm/IR/IRBuilder.h b/llvm/include/llvm/IR/IRBuilder.h --- a/llvm/include/llvm/IR/IRBuilder.h +++ b/llvm/include/llvm/IR/IRBuilder.h @@ -901,6 +901,25 @@ return CreateBinaryIntrinsic(Intrinsic::maximum, LHS, RHS, nullptr, Name); } + /// Create call to the get_fpenv intrinsic. + /// \param EnvTy Integer type used by target to represent floating-point + /// environment. + CallInst *createGetFPEnv(Type *EnvTy, const Twine &Name = "") { + return CreateIntrinsic(Intrinsic::get_fpenv, {EnvTy}, {}, nullptr, Name); + } + + /// Create call to the set_fpenv intrinsic. + /// \param FPEnv Pointer to Value representing the FP environment. + CallInst *createSetFPEnv(Value *FPEnv, const Twine &Name = "") { + return CreateIntrinsic(Intrinsic::set_fpenv, {FPEnv->getType()}, {FPEnv}, + nullptr, Name); + } + + /// Create call to the reset_fpenv intrinsic. + CallInst *createResetFPEnv(const Twine &Name = "") { + return CreateIntrinsic(Intrinsic::reset_fpenv, {}, {}, nullptr, Name); + } + private: /// Create a call to a masked intrinsic with given Id. CallInst *CreateMaskedIntrinsic(Intrinsic::ID Id, ArrayRef Ops, diff --git a/llvm/include/llvm/IR/Intrinsics.td b/llvm/include/llvm/IR/Intrinsics.td --- a/llvm/include/llvm/IR/Intrinsics.td +++ b/llvm/include/llvm/IR/Intrinsics.td @@ -665,6 +665,9 @@ let IntrProperties = [IntrInaccessibleMemOnly, IntrWillReturn] in { def int_flt_rounds : Intrinsic<[llvm_i32_ty], []>; + def int_get_fpenv : Intrinsic<[llvm_anyint_ty], []>; + def int_set_fpenv : Intrinsic<[], [llvm_anyint_ty]>; + def int_reset_fpenv : Intrinsic<[], []>; } //===--------------- Constrained Floating Point Intrinsics ----------------===// diff --git a/llvm/include/llvm/IR/RuntimeLibcalls.def b/llvm/include/llvm/IR/RuntimeLibcalls.def --- a/llvm/include/llvm/IR/RuntimeLibcalls.def +++ b/llvm/include/llvm/IR/RuntimeLibcalls.def @@ -280,6 +280,10 @@ HANDLE_LIBCALL(LLRINT_F128, "llrintl") HANDLE_LIBCALL(LLRINT_PPCF128, "llrintl") +// Floating point environment +HANDLE_LIBCALL(FEGETENV, "fegetenv") +HANDLE_LIBCALL(FESETENV, "fesetenv") + // Conversion HANDLE_LIBCALL(FPEXT_F32_PPCF128, "__gcc_stoq") HANDLE_LIBCALL(FPEXT_F64_PPCF128, "__gcc_dtoq") diff --git a/llvm/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp b/llvm/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp --- a/llvm/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp +++ b/llvm/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp @@ -187,6 +187,8 @@ SDValue ExpandInsertToVectorThroughStack(SDValue Op); SDValue ExpandVectorBuildThroughStack(SDNode* Node); + SDValue makeStateFunctionCall(SDNode *Node, RTLIB::Libcall LC, SDValue MemOp); + SDValue ExpandConstantFP(ConstantFPSDNode *CFP, bool UseCP); SDValue ExpandConstant(ConstantSDNode *CP); @@ -1449,6 +1451,30 @@ return DAG.getLoad(VT, dl, StoreChain, FIPtr, PtrInfo); } +// Helper used to make call to a library function that has one argument of +// pointer type. Return value is void or ignored. Such functions include +// 'fegetmode', 'fesetenv' and others, which are used to get or set state. +// Returns outgoing token chain. +SDValue SelectionDAGLegalize::makeStateFunctionCall(SDNode *Node, + RTLIB::Libcall LC, + SDValue MemOp) { + SDLoc DLoc(Node); + TargetLowering::ArgListTy Args; + TargetLowering::ArgListEntry Entry; + Entry.Node = MemOp; + Entry.Ty = MemOp.getValueType().getTypeForEVT(*DAG.getContext()); + Args.push_back(Entry); + SDValue Callee = DAG.getExternalSymbol(TLI.getLibcallName(LC), + TLI.getPointerTy(DAG.getDataLayout())); + TargetLowering::CallLoweringInfo CLI(DAG); + CLI.setDebugLoc(DLoc) + .setChain(Node->getOperand(0)) + .setLibCallee(TLI.getLibcallCallingConv(LC), + Type::getVoidTy(*DAG.getContext()), Callee, + std::move(Args)); + return TLI.LowerCallTo(CLI).second; +} + /// Bitcast a floating-point value to an integer value. Only bitcast the part /// containing the sign bit if the target has no integer value capable of /// holding all bits of the floating-point value. @@ -4348,6 +4374,43 @@ break; } break; + case ISD::GET_FPENV: { + // Call fegetenv, which saves environment into stack slot. Then load + // the value to return from the stack. + SDValue StackPtr = DAG.CreateStackTemporary(Node->getValueType(0)); + int SPFI = cast(StackPtr.getNode())->getIndex(); + SDValue Chain = makeStateFunctionCall(Node, RTLIB::FEGETENV, StackPtr); + SDValue LdInst = DAG.getLoad( + StackPtr.getValueType(), dl, Chain, StackPtr, + MachinePointerInfo::getFixedStack(DAG.getMachineFunction(), SPFI)); + Results.push_back(LdInst); + Results.push_back(LdInst.getValue(1)); + break; + } + case ISD::SET_FPENV: { + // Move environment to stack slot and then call fesetenv with the pointer + // to the slot as argument. + SDValue Env = Node->getOperand(1); + EVT ModeVT = Env.getValueType(); + SDValue StackPtr = DAG.CreateStackTemporary(ModeVT); + int SPFI = cast(StackPtr.getNode())->getIndex(); + SDValue StInst = DAG.getStore( + Node->getOperand(0), dl, Env, StackPtr, + MachinePointerInfo::getFixedStack(DAG.getMachineFunction(), SPFI)); + Results.push_back(makeStateFunctionCall(Node, RTLIB::FESETENV, StackPtr)); + break; + } + case ISD::RESET_FPENV: { + // It is legalized to call 'fesetenv(FE_DFL_ENV)'. On most targets + // FE_DFL_MODE is defined as '((const femode_t *) -1)' in glibc. + const DataLayout &DL = DAG.getDataLayout(); + Type *ByteTy = EVT(MVT::i8).getTypeForEVT(*DAG.getContext()); + auto *PtrTy = PointerType::get(ByteTy, DL.getAllocaAddrSpace()); + Type *IntTy = DL.getIntPtrType(PtrTy); + SDValue Env = DAG.getConstant(-1LL, dl, MVT::getVT(IntTy)); + Results.push_back(makeStateFunctionCall(Node, RTLIB::FESETENV, Env)); + break; + } } // Replace the original node with the legalized result. diff --git a/llvm/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp b/llvm/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp --- a/llvm/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp +++ b/llvm/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp @@ -2118,6 +2118,21 @@ case ISD::FSHR: ExpandIntRes_FunnelShift(N, Lo, Hi); break; + + case ISD::GET_FPENV: { + // We do need custom lowering. If size of FP environment is long enough + // (on X86 it is 256 bits), the node type is extended and getOperationAction + // unconditionally returns Expand for it. So make custom lowering here. + SmallVector Results; + TLI.ReplaceNodeResults(N, Results, DAG); + if (Results.empty()) + report_fatal_error("Do not know how to expand the result of this " + "operator!"); + assert(Results.size() == N->getNumValues() && + "Custom lowering returned the wrong number of results!"); + for (unsigned i = 0, e = Results.size(); i != e; ++i) + ReplaceValueWith(SDValue(N, i), Results[i]); + } } // If Lo/Hi is null, the sub-method took care of registering results etc. diff --git a/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp b/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp --- a/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp +++ b/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp @@ -6194,6 +6194,22 @@ DAG.getNode(ISD::BITCAST, sdl, MVT::f16, getValue(I.getArgOperand(0))))); return; + case Intrinsic::get_fpenv: + Res = DAG.getNode( + ISD::GET_FPENV, sdl, + DAG.getVTList(TLI.getValueType(DAG.getDataLayout(), I.getType()), + MVT::Other), + DAG.getRoot()); + setValue(&I, Res); + DAG.setRoot(Res.getValue(1)); + return; + case Intrinsic::set_fpenv: + DAG.setRoot(DAG.getNode(ISD::SET_FPENV, sdl, MVT::Other, getRoot(), + getValue(I.getArgOperand(0)))); + return; + case Intrinsic::reset_fpenv: + DAG.setRoot(DAG.getNode(ISD::RESET_FPENV, sdl, MVT::Other, getRoot())); + return; case Intrinsic::pcmarker: { SDValue Tmp = getValue(I.getArgOperand(0)); DAG.setRoot(DAG.getNode(ISD::PCMARKER, sdl, MVT::Other, getRoot(), Tmp)); diff --git a/llvm/lib/CodeGen/SelectionDAG/SelectionDAGDumper.cpp b/llvm/lib/CodeGen/SelectionDAG/SelectionDAGDumper.cpp --- a/llvm/lib/CodeGen/SelectionDAG/SelectionDAGDumper.cpp +++ b/llvm/lib/CodeGen/SelectionDAG/SelectionDAGDumper.cpp @@ -403,6 +403,11 @@ case ISD::PREALLOCATED_ARG: return "call_alloc"; + // Environment manipulation + case ISD::GET_FPENV: return "get_fpenv"; + case ISD::SET_FPENV: return "set_fpenv"; + case ISD::RESET_FPENV: return "reset_fpenv"; + // Bit manipulation case ISD::ABS: return "abs"; case ISD::BITREVERSE: return "bitreverse"; diff --git a/llvm/lib/CodeGen/TargetLoweringBase.cpp b/llvm/lib/CodeGen/TargetLoweringBase.cpp --- a/llvm/lib/CodeGen/TargetLoweringBase.cpp +++ b/llvm/lib/CodeGen/TargetLoweringBase.cpp @@ -775,6 +775,12 @@ // On most systems, DEBUGTRAP and TRAP have no difference. The "Expand" // here is to inform DAG Legalizer to replace DEBUGTRAP with TRAP. setOperationAction(ISD::DEBUGTRAP, MVT::Other, Expand); + + // FP environment operations default to expand to library calls. + for (MVT VT : {MVT::i16, MVT::i32, MVT::i64}) + setOperationAction(ISD::GET_FPENV, VT, Expand); + setOperationAction(ISD::SET_FPENV, MVT::Other, Expand); + setOperationAction(ISD::RESET_FPENV, MVT::Other, Expand); } MVT TargetLoweringBase::getScalarShiftAmountTy(const DataLayout &DL, diff --git a/llvm/test/CodeGen/Generic/fpenv.ll b/llvm/test/CodeGen/Generic/fpenv.ll new file mode 100644 --- /dev/null +++ b/llvm/test/CodeGen/Generic/fpenv.ll @@ -0,0 +1,53 @@ +; RUN: llc -mtriple=msp430 -o - < %s | FileCheck %s + +; This test checks default lowering of the intrinsics operating floating point +; environment. MSP430 is used as a target in this test because it does not have +; native FP support, so it hopefully won't get custom lowering for these +; intrinsics. +; +; REQUIRES: msp430-registered-target + +target datalayout = "e-p:16:16:16-i8:8:8-i16:16:16-i32:16:32-n8:16" +target triple = "msp430---elf" + + +define i16 @func_01() { +entry: + %fpenv = call i16 @llvm.get.fpenv.i16() + ret i16 %fpenv +} +; CHECK-LABEL: func_01: +; CHECK: sub #2, r1 +; CHECK-NEXT: mov r1, r12 +; CHECK-NEXT: call #fegetenv +; CHECK-NEXT: mov 0(r1), r12 +; CHECK-NEXT: add #2, r1 +; CHECK-NEXT: ret + + +define void @func_02(i16 %fpenv) { +entry: + call void @llvm.set.fpenv.i16(i16 %fpenv) + ret void +} +; CHECK-LABEL: func_02: +; CHECK: sub #2, r1 +; CHECK-NEXT: mov r1, r12 +; CHECK-NEXT: call #fesetenv +; CHECK-NEXT: add #2, r1 +; CHECK-NEXT: ret + + +define void @func_03() { +entry: + call void @llvm.reset.fpenv() + ret void +} +; CHECK-LABEL: func_03: +; CHECK: mov #-1, r12 +; CHECK-NEXT: call #fesetenv +; CHECK-NEXT: ret + +declare i16 @llvm.get.fpenv.i16() +declare void @llvm.set.fpenv.i16(i16 %fpenv) +declare void @llvm.reset.fpenv() diff --git a/llvm/unittests/IR/IRBuilderTest.cpp b/llvm/unittests/IR/IRBuilderTest.cpp --- a/llvm/unittests/IR/IRBuilderTest.cpp +++ b/llvm/unittests/IR/IRBuilderTest.cpp @@ -988,4 +988,24 @@ Builder.CreateAdd(Builder.getInt32(1), Builder.getInt32(2), "add"); EXPECT_EQ(Add->getName(), "add"); } + +TEST_F(IRBuilderTest, FPEnvironment) { + IRBuilder<> Builder(BB); + CallInst *Call; + IntrinsicInst *II; + + Call = Builder.createResetFPEnv(); + II = cast(Call); + EXPECT_EQ(Intrinsic::reset_fpenv, II->getIntrinsicID()); + + Call = Builder.createGetFPEnv(Builder.getInt32Ty()); + II = cast(Call); + EXPECT_EQ(Intrinsic::get_fpenv, II->getIntrinsicID()); + EXPECT_EQ(Call->getType(), Builder.getInt32Ty()); + + AllocaInst *Var1 = Builder.CreateAlloca(Builder.getInt32Ty()); + Call = Builder.createSetFPEnv(Var1); + II = cast(Call); + EXPECT_EQ(Intrinsic::set_fpenv, II->getIntrinsicID()); +} }