From: Saleem Abdulrasool Date: Thu, 23 May 2019 17:03:43 +0000 (+0000) Subject: Transforms: lower fadd and fsub atomicrmw instructions X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=b1cc82a2499e822690c89bfd9886d9ceaae16508;p=llvm Transforms: lower fadd and fsub atomicrmw instructions `fadd` and `fsub` have recently (r351850) been added as `atomicrmw` operations. This diff adds lowering cases for them to the LowerAtomic transform. Patch by Josh Berdine! git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@361512 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Transforms/Scalar/LowerAtomic.cpp b/lib/Transforms/Scalar/LowerAtomic.cpp index f39ca239644..e076424d904 100644 --- a/lib/Transforms/Scalar/LowerAtomic.cpp +++ b/lib/Transforms/Scalar/LowerAtomic.cpp @@ -86,6 +86,12 @@ static bool LowerAtomicRMWInst(AtomicRMWInst *RMWI) { Res = Builder.CreateSelect(Builder.CreateICmpULT(Orig, Val), Orig, Val); break; + case AtomicRMWInst::FAdd: + Res = Builder.CreateFAdd(Orig, Val); + break; + case AtomicRMWInst::FSub: + Res = Builder.CreateFSub(Orig, Val); + break; } Builder.CreateStore(Res, Ptr); RMWI->replaceAllUsesWith(Orig); diff --git a/test/Transforms/LowerAtomic/atomic-load.ll b/test/Transforms/LowerAtomic/atomic-load.ll index e73417f3d40..f4988c20b6a 100644 --- a/test/Transforms/LowerAtomic/atomic-load.ll +++ b/test/Transforms/LowerAtomic/atomic-load.ll @@ -35,3 +35,25 @@ define i8 @min() { ret i8 %j ; CHECK: ret i8 [[INST]] } + +define float @fadd() { +; CHECK-LABEL: @fadd( + %i = alloca float + %j = atomicrmw fadd float* %i, float 42.0 monotonic +; CHECK: [[INST:%[a-z0-9]+]] = load +; CHECK-NEXT: fadd +; CHECK-NEXT: store + ret float %j +; CHECK: ret float [[INST]] +} + +define float @fsub() { +; CHECK-LABEL: @fsub( + %i = alloca float + %j = atomicrmw fsub float* %i, float 42.0 monotonic +; CHECK: [[INST:%[a-z0-9]+]] = load +; CHECK-NEXT: fsub +; CHECK-NEXT: store + ret float %j +; CHECK: ret float [[INST]] +}