Robin Morisset | 59c23cd | 2014-08-21 21:50:01 +0000 | [diff] [blame] | 1 | //===-- AtomicExpandPass.cpp - Expand atomic instructions -------===// |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 2 | // |
| 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
| 5 | // This file is distributed under the University of Illinois Open Source |
| 6 | // License. See LICENSE.TXT for details. |
| 7 | // |
| 8 | //===----------------------------------------------------------------------===// |
| 9 | // |
| 10 | // This file contains a pass (at IR level) to replace atomic instructions with |
Philip Reames | 61a24ab | 2015-12-16 00:49:36 +0000 | [diff] [blame] | 11 | // target specific instruction which implement the same semantics in a way |
| 12 | // which better fits the target backend. This can include the use of either |
| 13 | // (intrinsic-based) load-linked/store-conditional loops, AtomicCmpXchg, or |
| 14 | // type coercions. |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 15 | // |
| 16 | //===----------------------------------------------------------------------===// |
| 17 | |
JF Bastien | e8aad29 | 2015-08-03 15:29:47 +0000 | [diff] [blame] | 18 | #include "llvm/CodeGen/AtomicExpandUtils.h" |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 19 | #include "llvm/CodeGen/Passes.h" |
| 20 | #include "llvm/IR/Function.h" |
| 21 | #include "llvm/IR/IRBuilder.h" |
Robin Morisset | ed3d48f | 2014-09-03 21:29:59 +0000 | [diff] [blame] | 22 | #include "llvm/IR/InstIterator.h" |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 23 | #include "llvm/IR/Instructions.h" |
| 24 | #include "llvm/IR/Intrinsics.h" |
| 25 | #include "llvm/IR/Module.h" |
| 26 | #include "llvm/Support/Debug.h" |
Philip Reames | 2331901 | 2015-12-16 01:24:05 +0000 | [diff] [blame] | 27 | #include "llvm/Support/raw_ostream.h" |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 28 | #include "llvm/Target/TargetLowering.h" |
| 29 | #include "llvm/Target/TargetMachine.h" |
Eric Christopher | c40e5ed | 2014-06-19 21:03:04 +0000 | [diff] [blame] | 30 | #include "llvm/Target/TargetSubtargetInfo.h" |
| 31 | |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 32 | using namespace llvm; |
| 33 | |
Robin Morisset | 59c23cd | 2014-08-21 21:50:01 +0000 | [diff] [blame] | 34 | #define DEBUG_TYPE "atomic-expand" |
Chandler Carruth | 1b9dde0 | 2014-04-22 02:02:50 +0000 | [diff] [blame] | 35 | |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 36 | namespace { |
Robin Morisset | 59c23cd | 2014-08-21 21:50:01 +0000 | [diff] [blame] | 37 | class AtomicExpand: public FunctionPass { |
Eric Christopher | c40e5ed | 2014-06-19 21:03:04 +0000 | [diff] [blame] | 38 | const TargetMachine *TM; |
Eric Christopher | b11a1b7 | 2015-01-26 19:45:40 +0000 | [diff] [blame] | 39 | const TargetLowering *TLI; |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 40 | public: |
| 41 | static char ID; // Pass identification, replacement for typeid |
Robin Morisset | 59c23cd | 2014-08-21 21:50:01 +0000 | [diff] [blame] | 42 | explicit AtomicExpand(const TargetMachine *TM = nullptr) |
Eric Christopher | b11a1b7 | 2015-01-26 19:45:40 +0000 | [diff] [blame] | 43 | : FunctionPass(ID), TM(TM), TLI(nullptr) { |
Robin Morisset | 59c23cd | 2014-08-21 21:50:01 +0000 | [diff] [blame] | 44 | initializeAtomicExpandPass(*PassRegistry::getPassRegistry()); |
Tim Northover | 037f26f2 | 2014-04-17 18:22:47 +0000 | [diff] [blame] | 45 | } |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 46 | |
| 47 | bool runOnFunction(Function &F) override; |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 48 | |
Robin Morisset | ed3d48f | 2014-09-03 21:29:59 +0000 | [diff] [blame] | 49 | private: |
Robin Morisset | dedef33 | 2014-09-23 20:31:14 +0000 | [diff] [blame] | 50 | bool bracketInstWithFences(Instruction *I, AtomicOrdering Order, |
| 51 | bool IsStore, bool IsLoad); |
Philip Reames | 61a24ab | 2015-12-16 00:49:36 +0000 | [diff] [blame] | 52 | IntegerType *getCorrespondingIntegerType(Type *T, const DataLayout &DL); |
| 53 | LoadInst *convertAtomicLoadToIntegerType(LoadInst *LI); |
Ahmed Bougacha | 5246867 | 2015-09-11 17:08:28 +0000 | [diff] [blame] | 54 | bool tryExpandAtomicLoad(LoadInst *LI); |
Robin Morisset | 6dbbbc2 | 2014-09-23 20:59:25 +0000 | [diff] [blame] | 55 | bool expandAtomicLoadToLL(LoadInst *LI); |
| 56 | bool expandAtomicLoadToCmpXchg(LoadInst *LI); |
Philip Reames | 61a24ab | 2015-12-16 00:49:36 +0000 | [diff] [blame] | 57 | StoreInst *convertAtomicStoreToIntegerType(StoreInst *SI); |
Robin Morisset | ed3d48f | 2014-09-03 21:29:59 +0000 | [diff] [blame] | 58 | bool expandAtomicStore(StoreInst *SI); |
JF Bastien | f14889e | 2015-03-04 15:47:57 +0000 | [diff] [blame] | 59 | bool tryExpandAtomicRMW(AtomicRMWInst *AI); |
Tim Northover | f520eff | 2015-12-02 18:12:57 +0000 | [diff] [blame] | 60 | bool expandAtomicOpToLLSC( |
| 61 | Instruction *I, Value *Addr, AtomicOrdering MemOpOrder, |
| 62 | std::function<Value *(IRBuilder<> &, Value *)> PerformOp); |
Philip Reames | 1960cfd | 2016-02-19 00:06:41 +0000 | [diff] [blame^] | 63 | AtomicCmpXchgInst *convertCmpXchgToIntegerType(AtomicCmpXchgInst *CI); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 64 | bool expandAtomicCmpXchg(AtomicCmpXchgInst *CI); |
Robin Morisset | 810739d | 2014-09-25 17:27:43 +0000 | [diff] [blame] | 65 | bool isIdempotentRMW(AtomicRMWInst *AI); |
| 66 | bool simplifyIdempotentRMW(AtomicRMWInst *AI); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 67 | }; |
Alexander Kornienko | f00654e | 2015-06-23 09:49:53 +0000 | [diff] [blame] | 68 | } |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 69 | |
Robin Morisset | 59c23cd | 2014-08-21 21:50:01 +0000 | [diff] [blame] | 70 | char AtomicExpand::ID = 0; |
| 71 | char &llvm::AtomicExpandID = AtomicExpand::ID; |
| 72 | INITIALIZE_TM_PASS(AtomicExpand, "atomic-expand", |
| 73 | "Expand Atomic calls in terms of either load-linked & store-conditional or cmpxchg", |
Jiangning Liu | d623c52 | 2014-06-11 07:04:37 +0000 | [diff] [blame] | 74 | false, false) |
Tim Northover | 037f26f2 | 2014-04-17 18:22:47 +0000 | [diff] [blame] | 75 | |
Robin Morisset | 59c23cd | 2014-08-21 21:50:01 +0000 | [diff] [blame] | 76 | FunctionPass *llvm::createAtomicExpandPass(const TargetMachine *TM) { |
| 77 | return new AtomicExpand(TM); |
Tim Northover | 037f26f2 | 2014-04-17 18:22:47 +0000 | [diff] [blame] | 78 | } |
| 79 | |
Robin Morisset | 59c23cd | 2014-08-21 21:50:01 +0000 | [diff] [blame] | 80 | bool AtomicExpand::runOnFunction(Function &F) { |
Eric Christopher | 4e048eeb | 2015-01-27 01:04:42 +0000 | [diff] [blame] | 81 | if (!TM || !TM->getSubtargetImpl(F)->enableAtomicExpand()) |
Tim Northover | 037f26f2 | 2014-04-17 18:22:47 +0000 | [diff] [blame] | 82 | return false; |
Eric Christopher | 4e048eeb | 2015-01-27 01:04:42 +0000 | [diff] [blame] | 83 | TLI = TM->getSubtargetImpl(F)->getTargetLowering(); |
Tim Northover | 037f26f2 | 2014-04-17 18:22:47 +0000 | [diff] [blame] | 84 | |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 85 | SmallVector<Instruction *, 1> AtomicInsts; |
| 86 | |
| 87 | // Changing control-flow while iterating through it is a bad idea, so gather a |
| 88 | // list of all atomic instructions before we start. |
Robin Morisset | ed3d48f | 2014-09-03 21:29:59 +0000 | [diff] [blame] | 89 | for (inst_iterator I = inst_begin(F), E = inst_end(F); I != E; ++I) { |
| 90 | if (I->isAtomic()) |
| 91 | AtomicInsts.push_back(&*I); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 92 | } |
| 93 | |
Robin Morisset | ed3d48f | 2014-09-03 21:29:59 +0000 | [diff] [blame] | 94 | bool MadeChange = false; |
| 95 | for (auto I : AtomicInsts) { |
| 96 | auto LI = dyn_cast<LoadInst>(I); |
| 97 | auto SI = dyn_cast<StoreInst>(I); |
| 98 | auto RMWI = dyn_cast<AtomicRMWInst>(I); |
| 99 | auto CASI = dyn_cast<AtomicCmpXchgInst>(I); |
Robin Morisset | ed3d48f | 2014-09-03 21:29:59 +0000 | [diff] [blame] | 100 | assert((LI || SI || RMWI || CASI || isa<FenceInst>(I)) && |
| 101 | "Unknown atomic instruction"); |
| 102 | |
Eric Christopher | b11a1b7 | 2015-01-26 19:45:40 +0000 | [diff] [blame] | 103 | if (TLI->getInsertFencesForAtomic()) { |
Philip Reames | 367fdd9 | 2016-02-18 19:45:31 +0000 | [diff] [blame] | 104 | auto FenceOrdering = Monotonic; |
| 105 | bool IsStore, IsLoad; |
Robin Morisset | dedef33 | 2014-09-23 20:31:14 +0000 | [diff] [blame] | 106 | if (LI && isAtLeastAcquire(LI->getOrdering())) { |
| 107 | FenceOrdering = LI->getOrdering(); |
| 108 | LI->setOrdering(Monotonic); |
| 109 | IsStore = false; |
| 110 | IsLoad = true; |
| 111 | } else if (SI && isAtLeastRelease(SI->getOrdering())) { |
| 112 | FenceOrdering = SI->getOrdering(); |
| 113 | SI->setOrdering(Monotonic); |
| 114 | IsStore = true; |
| 115 | IsLoad = false; |
| 116 | } else if (RMWI && (isAtLeastRelease(RMWI->getOrdering()) || |
| 117 | isAtLeastAcquire(RMWI->getOrdering()))) { |
| 118 | FenceOrdering = RMWI->getOrdering(); |
| 119 | RMWI->setOrdering(Monotonic); |
| 120 | IsStore = IsLoad = true; |
Ahmed Bougacha | 5246867 | 2015-09-11 17:08:28 +0000 | [diff] [blame] | 121 | } else if (CASI && !TLI->shouldExpandAtomicCmpXchgInIR(CASI) && |
Eric Christopher | b11a1b7 | 2015-01-26 19:45:40 +0000 | [diff] [blame] | 122 | (isAtLeastRelease(CASI->getSuccessOrdering()) || |
| 123 | isAtLeastAcquire(CASI->getSuccessOrdering()))) { |
Robin Morisset | dedef33 | 2014-09-23 20:31:14 +0000 | [diff] [blame] | 124 | // If a compare and swap is lowered to LL/SC, we can do smarter fence |
| 125 | // insertion, with a stronger one on the success path than on the |
| 126 | // failure path. As a result, fence insertion is directly done by |
| 127 | // expandAtomicCmpXchg in that case. |
| 128 | FenceOrdering = CASI->getSuccessOrdering(); |
| 129 | CASI->setSuccessOrdering(Monotonic); |
| 130 | CASI->setFailureOrdering(Monotonic); |
| 131 | IsStore = IsLoad = true; |
| 132 | } |
| 133 | |
| 134 | if (FenceOrdering != Monotonic) { |
| 135 | MadeChange |= bracketInstWithFences(I, FenceOrdering, IsStore, IsLoad); |
| 136 | } |
| 137 | } |
| 138 | |
Ahmed Bougacha | 5246867 | 2015-09-11 17:08:28 +0000 | [diff] [blame] | 139 | if (LI) { |
Philip Reames | 61a24ab | 2015-12-16 00:49:36 +0000 | [diff] [blame] | 140 | if (LI->getType()->isFloatingPointTy()) { |
| 141 | // TODO: add a TLI hook to control this so that each target can |
| 142 | // convert to lowering the original type one at a time. |
| 143 | LI = convertAtomicLoadToIntegerType(LI); |
| 144 | assert(LI->getType()->isIntegerTy() && "invariant broken"); |
| 145 | MadeChange = true; |
| 146 | } |
| 147 | |
Ahmed Bougacha | 5246867 | 2015-09-11 17:08:28 +0000 | [diff] [blame] | 148 | MadeChange |= tryExpandAtomicLoad(LI); |
Philip Reames | 61a24ab | 2015-12-16 00:49:36 +0000 | [diff] [blame] | 149 | } else if (SI) { |
| 150 | if (SI->getValueOperand()->getType()->isFloatingPointTy()) { |
| 151 | // TODO: add a TLI hook to control this so that each target can |
| 152 | // convert to lowering the original type one at a time. |
| 153 | SI = convertAtomicStoreToIntegerType(SI); |
| 154 | assert(SI->getValueOperand()->getType()->isIntegerTy() && |
| 155 | "invariant broken"); |
| 156 | MadeChange = true; |
| 157 | } |
| 158 | |
| 159 | if (TLI->shouldExpandAtomicStoreInIR(SI)) |
| 160 | MadeChange |= expandAtomicStore(SI); |
Robin Morisset | 810739d | 2014-09-25 17:27:43 +0000 | [diff] [blame] | 161 | } else if (RMWI) { |
| 162 | // There are two different ways of expanding RMW instructions: |
| 163 | // - into a load if it is idempotent |
| 164 | // - into a Cmpxchg/LL-SC loop otherwise |
| 165 | // we try them in that order. |
JF Bastien | f14889e | 2015-03-04 15:47:57 +0000 | [diff] [blame] | 166 | |
| 167 | if (isIdempotentRMW(RMWI) && simplifyIdempotentRMW(RMWI)) { |
| 168 | MadeChange = true; |
| 169 | } else { |
| 170 | MadeChange |= tryExpandAtomicRMW(RMWI); |
| 171 | } |
Philip Reames | 1960cfd | 2016-02-19 00:06:41 +0000 | [diff] [blame^] | 172 | } else if (CASI) { |
| 173 | // TODO: when we're ready to make the change at the IR level, we can |
| 174 | // extend convertCmpXchgToInteger for floating point too. |
| 175 | assert(!CASI->getCompareOperand()->getType()->isFloatingPointTy() && |
| 176 | "unimplemented - floating point not legal at IR level"); |
| 177 | if (CASI->getCompareOperand()->getType()->isPointerTy() ) { |
| 178 | // TODO: add a TLI hook to control this so that each target can |
| 179 | // convert to lowering the original type one at a time. |
| 180 | CASI = convertCmpXchgToIntegerType(CASI); |
| 181 | assert(CASI->getCompareOperand()->getType()->isIntegerTy() && |
| 182 | "invariant broken"); |
| 183 | MadeChange = true; |
| 184 | } |
| 185 | |
| 186 | if (TLI->shouldExpandAtomicCmpXchgInIR(CASI)) |
| 187 | MadeChange |= expandAtomicCmpXchg(CASI); |
Robin Morisset | ed3d48f | 2014-09-03 21:29:59 +0000 | [diff] [blame] | 188 | } |
| 189 | } |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 190 | return MadeChange; |
| 191 | } |
| 192 | |
Robin Morisset | dedef33 | 2014-09-23 20:31:14 +0000 | [diff] [blame] | 193 | bool AtomicExpand::bracketInstWithFences(Instruction *I, AtomicOrdering Order, |
| 194 | bool IsStore, bool IsLoad) { |
| 195 | IRBuilder<> Builder(I); |
| 196 | |
Eric Christopher | b11a1b7 | 2015-01-26 19:45:40 +0000 | [diff] [blame] | 197 | auto LeadingFence = TLI->emitLeadingFence(Builder, Order, IsStore, IsLoad); |
Robin Morisset | dedef33 | 2014-09-23 20:31:14 +0000 | [diff] [blame] | 198 | |
Eric Christopher | b11a1b7 | 2015-01-26 19:45:40 +0000 | [diff] [blame] | 199 | auto TrailingFence = TLI->emitTrailingFence(Builder, Order, IsStore, IsLoad); |
Robin Morisset | dedef33 | 2014-09-23 20:31:14 +0000 | [diff] [blame] | 200 | // The trailing fence is emitted before the instruction instead of after |
| 201 | // because there is no easy way of setting Builder insertion point after |
| 202 | // an instruction. So we must erase it from the BB, and insert it back |
| 203 | // in the right place. |
| 204 | // We have a guard here because not every atomic operation generates a |
| 205 | // trailing fence. |
| 206 | if (TrailingFence) { |
| 207 | TrailingFence->removeFromParent(); |
| 208 | TrailingFence->insertAfter(I); |
| 209 | } |
| 210 | |
| 211 | return (LeadingFence || TrailingFence); |
| 212 | } |
| 213 | |
Philip Reames | 61a24ab | 2015-12-16 00:49:36 +0000 | [diff] [blame] | 214 | /// Get the iX type with the same bitwidth as T. |
| 215 | IntegerType *AtomicExpand::getCorrespondingIntegerType(Type *T, |
| 216 | const DataLayout &DL) { |
| 217 | EVT VT = TLI->getValueType(DL, T); |
| 218 | unsigned BitWidth = VT.getStoreSizeInBits(); |
| 219 | assert(BitWidth == VT.getSizeInBits() && "must be a power of two"); |
| 220 | return IntegerType::get(T->getContext(), BitWidth); |
| 221 | } |
| 222 | |
| 223 | /// Convert an atomic load of a non-integral type to an integer load of the |
Philip Reames | 1960cfd | 2016-02-19 00:06:41 +0000 | [diff] [blame^] | 224 | /// equivalent bitwidth. See the function comment on |
Philip Reames | 61a24ab | 2015-12-16 00:49:36 +0000 | [diff] [blame] | 225 | /// convertAtomicStoreToIntegerType for background. |
| 226 | LoadInst *AtomicExpand::convertAtomicLoadToIntegerType(LoadInst *LI) { |
| 227 | auto *M = LI->getModule(); |
| 228 | Type *NewTy = getCorrespondingIntegerType(LI->getType(), |
| 229 | M->getDataLayout()); |
| 230 | |
| 231 | IRBuilder<> Builder(LI); |
| 232 | |
| 233 | Value *Addr = LI->getPointerOperand(); |
| 234 | Type *PT = PointerType::get(NewTy, |
| 235 | Addr->getType()->getPointerAddressSpace()); |
| 236 | Value *NewAddr = Builder.CreateBitCast(Addr, PT); |
| 237 | |
| 238 | auto *NewLI = Builder.CreateLoad(NewAddr); |
| 239 | NewLI->setAlignment(LI->getAlignment()); |
| 240 | NewLI->setVolatile(LI->isVolatile()); |
| 241 | NewLI->setAtomic(LI->getOrdering(), LI->getSynchScope()); |
| 242 | DEBUG(dbgs() << "Replaced " << *LI << " with " << *NewLI << "\n"); |
| 243 | |
| 244 | Value *NewVal = Builder.CreateBitCast(NewLI, LI->getType()); |
| 245 | LI->replaceAllUsesWith(NewVal); |
| 246 | LI->eraseFromParent(); |
| 247 | return NewLI; |
| 248 | } |
| 249 | |
Ahmed Bougacha | 5246867 | 2015-09-11 17:08:28 +0000 | [diff] [blame] | 250 | bool AtomicExpand::tryExpandAtomicLoad(LoadInst *LI) { |
| 251 | switch (TLI->shouldExpandAtomicLoadInIR(LI)) { |
| 252 | case TargetLoweringBase::AtomicExpansionKind::None: |
| 253 | return false; |
Tim Northover | f520eff | 2015-12-02 18:12:57 +0000 | [diff] [blame] | 254 | case TargetLoweringBase::AtomicExpansionKind::LLSC: |
| 255 | return expandAtomicOpToLLSC( |
| 256 | LI, LI->getPointerOperand(), LI->getOrdering(), |
| 257 | [](IRBuilder<> &Builder, Value *Loaded) { return Loaded; }); |
| 258 | case TargetLoweringBase::AtomicExpansionKind::LLOnly: |
Robin Morisset | 6dbbbc2 | 2014-09-23 20:59:25 +0000 | [diff] [blame] | 259 | return expandAtomicLoadToLL(LI); |
Tim Northover | f520eff | 2015-12-02 18:12:57 +0000 | [diff] [blame] | 260 | case TargetLoweringBase::AtomicExpansionKind::CmpXChg: |
Robin Morisset | 6dbbbc2 | 2014-09-23 20:59:25 +0000 | [diff] [blame] | 261 | return expandAtomicLoadToCmpXchg(LI); |
Ahmed Bougacha | 5246867 | 2015-09-11 17:08:28 +0000 | [diff] [blame] | 262 | } |
Ahmed Bougacha | 5246867 | 2015-09-11 17:08:28 +0000 | [diff] [blame] | 263 | llvm_unreachable("Unhandled case in tryExpandAtomicLoad"); |
Robin Morisset | 6dbbbc2 | 2014-09-23 20:59:25 +0000 | [diff] [blame] | 264 | } |
| 265 | |
| 266 | bool AtomicExpand::expandAtomicLoadToLL(LoadInst *LI) { |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 267 | IRBuilder<> Builder(LI); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 268 | |
Robin Morisset | dedef33 | 2014-09-23 20:31:14 +0000 | [diff] [blame] | 269 | // On some architectures, load-linked instructions are atomic for larger |
| 270 | // sizes than normal loads. For example, the only 64-bit load guaranteed |
| 271 | // to be single-copy atomic by ARM is an ldrexd (A3.5.3). |
Robin Morisset | a47cb41 | 2014-09-03 21:01:03 +0000 | [diff] [blame] | 272 | Value *Val = |
Robin Morisset | dedef33 | 2014-09-23 20:31:14 +0000 | [diff] [blame] | 273 | TLI->emitLoadLinked(Builder, LI->getPointerOperand(), LI->getOrdering()); |
Tim Northover | f520eff | 2015-12-02 18:12:57 +0000 | [diff] [blame] | 274 | TLI->emitAtomicCmpXchgNoStoreLLBalance(Builder); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 275 | |
| 276 | LI->replaceAllUsesWith(Val); |
| 277 | LI->eraseFromParent(); |
| 278 | |
| 279 | return true; |
| 280 | } |
| 281 | |
Robin Morisset | 6dbbbc2 | 2014-09-23 20:59:25 +0000 | [diff] [blame] | 282 | bool AtomicExpand::expandAtomicLoadToCmpXchg(LoadInst *LI) { |
| 283 | IRBuilder<> Builder(LI); |
| 284 | AtomicOrdering Order = LI->getOrdering(); |
| 285 | Value *Addr = LI->getPointerOperand(); |
| 286 | Type *Ty = cast<PointerType>(Addr->getType())->getElementType(); |
| 287 | Constant *DummyVal = Constant::getNullValue(Ty); |
| 288 | |
| 289 | Value *Pair = Builder.CreateAtomicCmpXchg( |
| 290 | Addr, DummyVal, DummyVal, Order, |
| 291 | AtomicCmpXchgInst::getStrongestFailureOrdering(Order)); |
| 292 | Value *Loaded = Builder.CreateExtractValue(Pair, 0, "loaded"); |
| 293 | |
| 294 | LI->replaceAllUsesWith(Loaded); |
| 295 | LI->eraseFromParent(); |
| 296 | |
| 297 | return true; |
| 298 | } |
| 299 | |
Philip Reames | 61a24ab | 2015-12-16 00:49:36 +0000 | [diff] [blame] | 300 | /// Convert an atomic store of a non-integral type to an integer store of the |
Philip Reames | 1960cfd | 2016-02-19 00:06:41 +0000 | [diff] [blame^] | 301 | /// equivalent bitwidth. We used to not support floating point or vector |
Philip Reames | 61a24ab | 2015-12-16 00:49:36 +0000 | [diff] [blame] | 302 | /// atomics in the IR at all. The backends learned to deal with the bitcast |
| 303 | /// idiom because that was the only way of expressing the notion of a atomic |
| 304 | /// float or vector store. The long term plan is to teach each backend to |
| 305 | /// instruction select from the original atomic store, but as a migration |
| 306 | /// mechanism, we convert back to the old format which the backends understand. |
| 307 | /// Each backend will need individual work to recognize the new format. |
| 308 | StoreInst *AtomicExpand::convertAtomicStoreToIntegerType(StoreInst *SI) { |
| 309 | IRBuilder<> Builder(SI); |
| 310 | auto *M = SI->getModule(); |
| 311 | Type *NewTy = getCorrespondingIntegerType(SI->getValueOperand()->getType(), |
| 312 | M->getDataLayout()); |
| 313 | Value *NewVal = Builder.CreateBitCast(SI->getValueOperand(), NewTy); |
| 314 | |
| 315 | Value *Addr = SI->getPointerOperand(); |
| 316 | Type *PT = PointerType::get(NewTy, |
| 317 | Addr->getType()->getPointerAddressSpace()); |
| 318 | Value *NewAddr = Builder.CreateBitCast(Addr, PT); |
| 319 | |
| 320 | StoreInst *NewSI = Builder.CreateStore(NewVal, NewAddr); |
| 321 | NewSI->setAlignment(SI->getAlignment()); |
| 322 | NewSI->setVolatile(SI->isVolatile()); |
| 323 | NewSI->setAtomic(SI->getOrdering(), SI->getSynchScope()); |
| 324 | DEBUG(dbgs() << "Replaced " << *SI << " with " << *NewSI << "\n"); |
| 325 | SI->eraseFromParent(); |
| 326 | return NewSI; |
| 327 | } |
| 328 | |
Robin Morisset | 59c23cd | 2014-08-21 21:50:01 +0000 | [diff] [blame] | 329 | bool AtomicExpand::expandAtomicStore(StoreInst *SI) { |
Robin Morisset | 25c8e31 | 2014-09-17 00:06:58 +0000 | [diff] [blame] | 330 | // This function is only called on atomic stores that are too large to be |
| 331 | // atomic if implemented as a native store. So we replace them by an |
| 332 | // atomic swap, that can be implemented for example as a ldrex/strex on ARM |
| 333 | // or lock cmpxchg8/16b on X86, as these are atomic for larger sizes. |
JF Bastien | f14889e | 2015-03-04 15:47:57 +0000 | [diff] [blame] | 334 | // It is the responsibility of the target to only signal expansion via |
Robin Morisset | 25c8e31 | 2014-09-17 00:06:58 +0000 | [diff] [blame] | 335 | // shouldExpandAtomicRMW in cases where this is required and possible. |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 336 | IRBuilder<> Builder(SI); |
| 337 | AtomicRMWInst *AI = |
| 338 | Builder.CreateAtomicRMW(AtomicRMWInst::Xchg, SI->getPointerOperand(), |
| 339 | SI->getValueOperand(), SI->getOrdering()); |
| 340 | SI->eraseFromParent(); |
| 341 | |
| 342 | // Now we have an appropriate swap instruction, lower it as usual. |
JF Bastien | f14889e | 2015-03-04 15:47:57 +0000 | [diff] [blame] | 343 | return tryExpandAtomicRMW(AI); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 344 | } |
| 345 | |
JF Bastien | e8aad29 | 2015-08-03 15:29:47 +0000 | [diff] [blame] | 346 | static void createCmpXchgInstFun(IRBuilder<> &Builder, Value *Addr, |
| 347 | Value *Loaded, Value *NewVal, |
| 348 | AtomicOrdering MemOpOrder, |
| 349 | Value *&Success, Value *&NewLoaded) { |
| 350 | Value* Pair = Builder.CreateAtomicCmpXchg( |
| 351 | Addr, Loaded, NewVal, MemOpOrder, |
| 352 | AtomicCmpXchgInst::getStrongestFailureOrdering(MemOpOrder)); |
| 353 | Success = Builder.CreateExtractValue(Pair, 1, "success"); |
| 354 | NewLoaded = Builder.CreateExtractValue(Pair, 0, "newloaded"); |
| 355 | } |
| 356 | |
Robin Morisset | 25c8e31 | 2014-09-17 00:06:58 +0000 | [diff] [blame] | 357 | /// Emit IR to implement the given atomicrmw operation on values in registers, |
| 358 | /// returning the new value. |
| 359 | static Value *performAtomicOp(AtomicRMWInst::BinOp Op, IRBuilder<> &Builder, |
| 360 | Value *Loaded, Value *Inc) { |
| 361 | Value *NewVal; |
| 362 | switch (Op) { |
| 363 | case AtomicRMWInst::Xchg: |
| 364 | return Inc; |
| 365 | case AtomicRMWInst::Add: |
| 366 | return Builder.CreateAdd(Loaded, Inc, "new"); |
| 367 | case AtomicRMWInst::Sub: |
| 368 | return Builder.CreateSub(Loaded, Inc, "new"); |
| 369 | case AtomicRMWInst::And: |
| 370 | return Builder.CreateAnd(Loaded, Inc, "new"); |
| 371 | case AtomicRMWInst::Nand: |
| 372 | return Builder.CreateNot(Builder.CreateAnd(Loaded, Inc), "new"); |
| 373 | case AtomicRMWInst::Or: |
| 374 | return Builder.CreateOr(Loaded, Inc, "new"); |
| 375 | case AtomicRMWInst::Xor: |
| 376 | return Builder.CreateXor(Loaded, Inc, "new"); |
| 377 | case AtomicRMWInst::Max: |
| 378 | NewVal = Builder.CreateICmpSGT(Loaded, Inc); |
| 379 | return Builder.CreateSelect(NewVal, Loaded, Inc, "new"); |
| 380 | case AtomicRMWInst::Min: |
| 381 | NewVal = Builder.CreateICmpSLE(Loaded, Inc); |
| 382 | return Builder.CreateSelect(NewVal, Loaded, Inc, "new"); |
| 383 | case AtomicRMWInst::UMax: |
| 384 | NewVal = Builder.CreateICmpUGT(Loaded, Inc); |
| 385 | return Builder.CreateSelect(NewVal, Loaded, Inc, "new"); |
| 386 | case AtomicRMWInst::UMin: |
| 387 | NewVal = Builder.CreateICmpULE(Loaded, Inc); |
| 388 | return Builder.CreateSelect(NewVal, Loaded, Inc, "new"); |
| 389 | default: |
| 390 | llvm_unreachable("Unknown atomic op"); |
| 391 | } |
| 392 | } |
| 393 | |
Tim Northover | f520eff | 2015-12-02 18:12:57 +0000 | [diff] [blame] | 394 | bool AtomicExpand::tryExpandAtomicRMW(AtomicRMWInst *AI) { |
| 395 | switch (TLI->shouldExpandAtomicRMWInIR(AI)) { |
| 396 | case TargetLoweringBase::AtomicExpansionKind::None: |
| 397 | return false; |
| 398 | case TargetLoweringBase::AtomicExpansionKind::LLSC: |
| 399 | return expandAtomicOpToLLSC(AI, AI->getPointerOperand(), AI->getOrdering(), |
| 400 | [&](IRBuilder<> &Builder, Value *Loaded) { |
| 401 | return performAtomicOp(AI->getOperation(), |
| 402 | Builder, Loaded, |
| 403 | AI->getValOperand()); |
| 404 | }); |
| 405 | case TargetLoweringBase::AtomicExpansionKind::CmpXChg: |
| 406 | return expandAtomicRMWToCmpXchg(AI, createCmpXchgInstFun); |
| 407 | default: |
| 408 | llvm_unreachable("Unhandled case in tryExpandAtomicRMW"); |
| 409 | } |
| 410 | } |
| 411 | |
| 412 | bool AtomicExpand::expandAtomicOpToLLSC( |
| 413 | Instruction *I, Value *Addr, AtomicOrdering MemOpOrder, |
| 414 | std::function<Value *(IRBuilder<> &, Value *)> PerformOp) { |
| 415 | BasicBlock *BB = I->getParent(); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 416 | Function *F = BB->getParent(); |
| 417 | LLVMContext &Ctx = F->getContext(); |
| 418 | |
| 419 | // Given: atomicrmw some_op iN* %addr, iN %incr ordering |
| 420 | // |
| 421 | // The standard expansion we produce is: |
| 422 | // [...] |
| 423 | // fence? |
| 424 | // atomicrmw.start: |
| 425 | // %loaded = @load.linked(%addr) |
| 426 | // %new = some_op iN %loaded, %incr |
| 427 | // %stored = @store_conditional(%new, %addr) |
| 428 | // %try_again = icmp i32 ne %stored, 0 |
| 429 | // br i1 %try_again, label %loop, label %atomicrmw.end |
| 430 | // atomicrmw.end: |
| 431 | // fence? |
| 432 | // [...] |
Tim Northover | f520eff | 2015-12-02 18:12:57 +0000 | [diff] [blame] | 433 | BasicBlock *ExitBB = BB->splitBasicBlock(I->getIterator(), "atomicrmw.end"); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 434 | BasicBlock *LoopBB = BasicBlock::Create(Ctx, "atomicrmw.start", F, ExitBB); |
| 435 | |
Tim Northover | f520eff | 2015-12-02 18:12:57 +0000 | [diff] [blame] | 436 | // This grabs the DebugLoc from I. |
| 437 | IRBuilder<> Builder(I); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 438 | |
| 439 | // The split call above "helpfully" added a branch at the end of BB (to the |
| 440 | // wrong place), but we might want a fence too. It's easiest to just remove |
| 441 | // the branch entirely. |
| 442 | std::prev(BB->end())->eraseFromParent(); |
| 443 | Builder.SetInsertPoint(BB); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 444 | Builder.CreateBr(LoopBB); |
| 445 | |
| 446 | // Start the main loop block now that we've taken care of the preliminaries. |
| 447 | Builder.SetInsertPoint(LoopBB); |
Robin Morisset | a47cb41 | 2014-09-03 21:01:03 +0000 | [diff] [blame] | 448 | Value *Loaded = TLI->emitLoadLinked(Builder, Addr, MemOpOrder); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 449 | |
Tim Northover | f520eff | 2015-12-02 18:12:57 +0000 | [diff] [blame] | 450 | Value *NewVal = PerformOp(Builder, Loaded); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 451 | |
Eric Christopher | d913448 | 2014-08-04 21:25:23 +0000 | [diff] [blame] | 452 | Value *StoreSuccess = |
Robin Morisset | a47cb41 | 2014-09-03 21:01:03 +0000 | [diff] [blame] | 453 | TLI->emitStoreConditional(Builder, NewVal, Addr, MemOpOrder); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 454 | Value *TryAgain = Builder.CreateICmpNE( |
| 455 | StoreSuccess, ConstantInt::get(IntegerType::get(Ctx, 32), 0), "tryagain"); |
| 456 | Builder.CreateCondBr(TryAgain, LoopBB, ExitBB); |
| 457 | |
| 458 | Builder.SetInsertPoint(ExitBB, ExitBB->begin()); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 459 | |
Tim Northover | f520eff | 2015-12-02 18:12:57 +0000 | [diff] [blame] | 460 | I->replaceAllUsesWith(Loaded); |
| 461 | I->eraseFromParent(); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 462 | |
| 463 | return true; |
| 464 | } |
| 465 | |
Philip Reames | 1960cfd | 2016-02-19 00:06:41 +0000 | [diff] [blame^] | 466 | /// Convert an atomic cmpxchg of a non-integral type to an integer cmpxchg of |
| 467 | /// the equivalent bitwidth. We used to not support pointer cmpxchg in the |
| 468 | /// IR. As a migration step, we convert back to what use to be the standard |
| 469 | /// way to represent a pointer cmpxchg so that we can update backends one by |
| 470 | /// one. |
| 471 | AtomicCmpXchgInst *AtomicExpand::convertCmpXchgToIntegerType(AtomicCmpXchgInst *CI) { |
| 472 | auto *M = CI->getModule(); |
| 473 | Type *NewTy = getCorrespondingIntegerType(CI->getCompareOperand()->getType(), |
| 474 | M->getDataLayout()); |
| 475 | |
| 476 | IRBuilder<> Builder(CI); |
| 477 | |
| 478 | Value *Addr = CI->getPointerOperand(); |
| 479 | Type *PT = PointerType::get(NewTy, |
| 480 | Addr->getType()->getPointerAddressSpace()); |
| 481 | Value *NewAddr = Builder.CreateBitCast(Addr, PT); |
| 482 | |
| 483 | Value *NewCmp = Builder.CreatePtrToInt(CI->getCompareOperand(), NewTy); |
| 484 | Value *NewNewVal = Builder.CreatePtrToInt(CI->getNewValOperand(), NewTy); |
| 485 | |
| 486 | |
| 487 | auto *NewCI = Builder.CreateAtomicCmpXchg(NewAddr, NewCmp, NewNewVal, |
| 488 | CI->getSuccessOrdering(), |
| 489 | CI->getFailureOrdering(), |
| 490 | CI->getSynchScope()); |
| 491 | NewCI->setVolatile(CI->isVolatile()); |
| 492 | NewCI->setWeak(CI->isWeak()); |
| 493 | DEBUG(dbgs() << "Replaced " << *CI << " with " << *NewCI << "\n"); |
| 494 | |
| 495 | Value *OldVal = Builder.CreateExtractValue(NewCI, 0); |
| 496 | Value *Succ = Builder.CreateExtractValue(NewCI, 1); |
| 497 | |
| 498 | OldVal = Builder.CreateIntToPtr(OldVal, CI->getCompareOperand()->getType()); |
| 499 | |
| 500 | Value *Res = UndefValue::get(CI->getType()); |
| 501 | Res = Builder.CreateInsertValue(Res, OldVal, 0); |
| 502 | Res = Builder.CreateInsertValue(Res, Succ, 1); |
| 503 | |
| 504 | CI->replaceAllUsesWith(Res); |
| 505 | CI->eraseFromParent(); |
| 506 | return NewCI; |
| 507 | } |
| 508 | |
| 509 | |
Robin Morisset | 59c23cd | 2014-08-21 21:50:01 +0000 | [diff] [blame] | 510 | bool AtomicExpand::expandAtomicCmpXchg(AtomicCmpXchgInst *CI) { |
Tim Northover | 70450c5 | 2014-04-03 13:06:54 +0000 | [diff] [blame] | 511 | AtomicOrdering SuccessOrder = CI->getSuccessOrdering(); |
| 512 | AtomicOrdering FailureOrder = CI->getFailureOrdering(); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 513 | Value *Addr = CI->getPointerOperand(); |
| 514 | BasicBlock *BB = CI->getParent(); |
| 515 | Function *F = BB->getParent(); |
| 516 | LLVMContext &Ctx = F->getContext(); |
Robin Morisset | ed3d48f | 2014-09-03 21:29:59 +0000 | [diff] [blame] | 517 | // If getInsertFencesForAtomic() returns true, then the target does not want |
| 518 | // to deal with memory orders, and emitLeading/TrailingFence should take care |
| 519 | // of everything. Otherwise, emitLeading/TrailingFence are no-op and we |
| 520 | // should preserve the ordering. |
Robin Morisset | a47cb41 | 2014-09-03 21:01:03 +0000 | [diff] [blame] | 521 | AtomicOrdering MemOpOrder = |
| 522 | TLI->getInsertFencesForAtomic() ? Monotonic : SuccessOrder; |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 523 | |
| 524 | // Given: cmpxchg some_op iN* %addr, iN %desired, iN %new success_ord fail_ord |
| 525 | // |
Tim Northover | 70450c5 | 2014-04-03 13:06:54 +0000 | [diff] [blame] | 526 | // The full expansion we produce is: |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 527 | // [...] |
| 528 | // fence? |
| 529 | // cmpxchg.start: |
| 530 | // %loaded = @load.linked(%addr) |
| 531 | // %should_store = icmp eq %loaded, %desired |
Tim Northover | 70450c5 | 2014-04-03 13:06:54 +0000 | [diff] [blame] | 532 | // br i1 %should_store, label %cmpxchg.trystore, |
Ahmed Bougacha | 07a844d | 2015-09-22 17:21:44 +0000 | [diff] [blame] | 533 | // label %cmpxchg.nostore |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 534 | // cmpxchg.trystore: |
| 535 | // %stored = @store_conditional(%new, %addr) |
Tim Northover | 20b9f73 | 2014-06-13 16:45:52 +0000 | [diff] [blame] | 536 | // %success = icmp eq i32 %stored, 0 |
| 537 | // br i1 %success, label %cmpxchg.success, label %loop/%cmpxchg.failure |
| 538 | // cmpxchg.success: |
| 539 | // fence? |
| 540 | // br label %cmpxchg.end |
Ahmed Bougacha | 07a844d | 2015-09-22 17:21:44 +0000 | [diff] [blame] | 541 | // cmpxchg.nostore: |
| 542 | // @load_linked_fail_balance()? |
| 543 | // br label %cmpxchg.failure |
Tim Northover | 20b9f73 | 2014-06-13 16:45:52 +0000 | [diff] [blame] | 544 | // cmpxchg.failure: |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 545 | // fence? |
Tim Northover | 70450c5 | 2014-04-03 13:06:54 +0000 | [diff] [blame] | 546 | // br label %cmpxchg.end |
| 547 | // cmpxchg.end: |
Tim Northover | 20b9f73 | 2014-06-13 16:45:52 +0000 | [diff] [blame] | 548 | // %success = phi i1 [true, %cmpxchg.success], [false, %cmpxchg.failure] |
| 549 | // %restmp = insertvalue { iN, i1 } undef, iN %loaded, 0 |
| 550 | // %res = insertvalue { iN, i1 } %restmp, i1 %success, 1 |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 551 | // [...] |
Duncan P. N. Exon Smith | 8f11e1a | 2015-10-09 16:54:49 +0000 | [diff] [blame] | 552 | BasicBlock *ExitBB = BB->splitBasicBlock(CI->getIterator(), "cmpxchg.end"); |
Tim Northover | 20b9f73 | 2014-06-13 16:45:52 +0000 | [diff] [blame] | 553 | auto FailureBB = BasicBlock::Create(Ctx, "cmpxchg.failure", F, ExitBB); |
Ahmed Bougacha | 07a844d | 2015-09-22 17:21:44 +0000 | [diff] [blame] | 554 | auto NoStoreBB = BasicBlock::Create(Ctx, "cmpxchg.nostore", F, FailureBB); |
| 555 | auto SuccessBB = BasicBlock::Create(Ctx, "cmpxchg.success", F, NoStoreBB); |
Tim Northover | 20b9f73 | 2014-06-13 16:45:52 +0000 | [diff] [blame] | 556 | auto TryStoreBB = BasicBlock::Create(Ctx, "cmpxchg.trystore", F, SuccessBB); |
Tim Northover | 70450c5 | 2014-04-03 13:06:54 +0000 | [diff] [blame] | 557 | auto LoopBB = BasicBlock::Create(Ctx, "cmpxchg.start", F, TryStoreBB); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 558 | |
| 559 | // This grabs the DebugLoc from CI |
| 560 | IRBuilder<> Builder(CI); |
| 561 | |
| 562 | // The split call above "helpfully" added a branch at the end of BB (to the |
| 563 | // wrong place), but we might want a fence too. It's easiest to just remove |
| 564 | // the branch entirely. |
| 565 | std::prev(BB->end())->eraseFromParent(); |
| 566 | Builder.SetInsertPoint(BB); |
Robin Morisset | a47cb41 | 2014-09-03 21:01:03 +0000 | [diff] [blame] | 567 | TLI->emitLeadingFence(Builder, SuccessOrder, /*IsStore=*/true, |
| 568 | /*IsLoad=*/true); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 569 | Builder.CreateBr(LoopBB); |
| 570 | |
| 571 | // Start the main loop block now that we've taken care of the preliminaries. |
| 572 | Builder.SetInsertPoint(LoopBB); |
Robin Morisset | a47cb41 | 2014-09-03 21:01:03 +0000 | [diff] [blame] | 573 | Value *Loaded = TLI->emitLoadLinked(Builder, Addr, MemOpOrder); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 574 | Value *ShouldStore = |
| 575 | Builder.CreateICmpEQ(Loaded, CI->getCompareOperand(), "should_store"); |
Tim Northover | 70450c5 | 2014-04-03 13:06:54 +0000 | [diff] [blame] | 576 | |
Eric Christopher | 572e03a | 2015-06-19 01:53:21 +0000 | [diff] [blame] | 577 | // If the cmpxchg doesn't actually need any ordering when it fails, we can |
Tim Northover | 70450c5 | 2014-04-03 13:06:54 +0000 | [diff] [blame] | 578 | // jump straight past that fence instruction (if it exists). |
Ahmed Bougacha | 07a844d | 2015-09-22 17:21:44 +0000 | [diff] [blame] | 579 | Builder.CreateCondBr(ShouldStore, TryStoreBB, NoStoreBB); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 580 | |
| 581 | Builder.SetInsertPoint(TryStoreBB); |
Robin Morisset | a47cb41 | 2014-09-03 21:01:03 +0000 | [diff] [blame] | 582 | Value *StoreSuccess = TLI->emitStoreConditional( |
| 583 | Builder, CI->getNewValOperand(), Addr, MemOpOrder); |
Tim Northover | d039abd | 2014-06-13 16:45:36 +0000 | [diff] [blame] | 584 | StoreSuccess = Builder.CreateICmpEQ( |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 585 | StoreSuccess, ConstantInt::get(Type::getInt32Ty(Ctx), 0), "success"); |
Tim Northover | 20b9f73 | 2014-06-13 16:45:52 +0000 | [diff] [blame] | 586 | Builder.CreateCondBr(StoreSuccess, SuccessBB, |
| 587 | CI->isWeak() ? FailureBB : LoopBB); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 588 | |
Tim Northover | b4ddc08 | 2014-05-30 10:09:59 +0000 | [diff] [blame] | 589 | // Make sure later instructions don't get reordered with a fence if necessary. |
Tim Northover | 20b9f73 | 2014-06-13 16:45:52 +0000 | [diff] [blame] | 590 | Builder.SetInsertPoint(SuccessBB); |
Robin Morisset | a47cb41 | 2014-09-03 21:01:03 +0000 | [diff] [blame] | 591 | TLI->emitTrailingFence(Builder, SuccessOrder, /*IsStore=*/true, |
| 592 | /*IsLoad=*/true); |
Tim Northover | 70450c5 | 2014-04-03 13:06:54 +0000 | [diff] [blame] | 593 | Builder.CreateBr(ExitBB); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 594 | |
Ahmed Bougacha | 07a844d | 2015-09-22 17:21:44 +0000 | [diff] [blame] | 595 | Builder.SetInsertPoint(NoStoreBB); |
| 596 | // In the failing case, where we don't execute the store-conditional, the |
| 597 | // target might want to balance out the load-linked with a dedicated |
| 598 | // instruction (e.g., on ARM, clearing the exclusive monitor). |
| 599 | TLI->emitAtomicCmpXchgNoStoreLLBalance(Builder); |
| 600 | Builder.CreateBr(FailureBB); |
| 601 | |
Tim Northover | 20b9f73 | 2014-06-13 16:45:52 +0000 | [diff] [blame] | 602 | Builder.SetInsertPoint(FailureBB); |
Robin Morisset | a47cb41 | 2014-09-03 21:01:03 +0000 | [diff] [blame] | 603 | TLI->emitTrailingFence(Builder, FailureOrder, /*IsStore=*/true, |
| 604 | /*IsLoad=*/true); |
Tim Northover | 20b9f73 | 2014-06-13 16:45:52 +0000 | [diff] [blame] | 605 | Builder.CreateBr(ExitBB); |
| 606 | |
Tim Northover | b4ddc08 | 2014-05-30 10:09:59 +0000 | [diff] [blame] | 607 | // Finally, we have control-flow based knowledge of whether the cmpxchg |
| 608 | // succeeded or not. We expose this to later passes by converting any |
| 609 | // subsequent "icmp eq/ne %loaded, %oldval" into a use of an appropriate PHI. |
| 610 | |
| 611 | // Setup the builder so we can create any PHIs we need. |
Tim Northover | 20b9f73 | 2014-06-13 16:45:52 +0000 | [diff] [blame] | 612 | Builder.SetInsertPoint(ExitBB, ExitBB->begin()); |
Tim Northover | 420a216 | 2014-06-13 14:24:07 +0000 | [diff] [blame] | 613 | PHINode *Success = Builder.CreatePHI(Type::getInt1Ty(Ctx), 2); |
| 614 | Success->addIncoming(ConstantInt::getTrue(Ctx), SuccessBB); |
Tim Northover | 20b9f73 | 2014-06-13 16:45:52 +0000 | [diff] [blame] | 615 | Success->addIncoming(ConstantInt::getFalse(Ctx), FailureBB); |
Tim Northover | b4ddc08 | 2014-05-30 10:09:59 +0000 | [diff] [blame] | 616 | |
| 617 | // Look for any users of the cmpxchg that are just comparing the loaded value |
| 618 | // against the desired one, and replace them with the CFG-derived version. |
Tim Northover | 420a216 | 2014-06-13 14:24:07 +0000 | [diff] [blame] | 619 | SmallVector<ExtractValueInst *, 2> PrunedInsts; |
Tim Northover | b4ddc08 | 2014-05-30 10:09:59 +0000 | [diff] [blame] | 620 | for (auto User : CI->users()) { |
Tim Northover | 420a216 | 2014-06-13 14:24:07 +0000 | [diff] [blame] | 621 | ExtractValueInst *EV = dyn_cast<ExtractValueInst>(User); |
| 622 | if (!EV) |
Tim Northover | b4ddc08 | 2014-05-30 10:09:59 +0000 | [diff] [blame] | 623 | continue; |
| 624 | |
Tim Northover | 420a216 | 2014-06-13 14:24:07 +0000 | [diff] [blame] | 625 | assert(EV->getNumIndices() == 1 && EV->getIndices()[0] <= 1 && |
| 626 | "weird extraction from { iN, i1 }"); |
Tim Northover | b4ddc08 | 2014-05-30 10:09:59 +0000 | [diff] [blame] | 627 | |
Tim Northover | 420a216 | 2014-06-13 14:24:07 +0000 | [diff] [blame] | 628 | if (EV->getIndices()[0] == 0) |
| 629 | EV->replaceAllUsesWith(Loaded); |
| 630 | else |
| 631 | EV->replaceAllUsesWith(Success); |
| 632 | |
| 633 | PrunedInsts.push_back(EV); |
Tim Northover | b4ddc08 | 2014-05-30 10:09:59 +0000 | [diff] [blame] | 634 | } |
| 635 | |
Tim Northover | 420a216 | 2014-06-13 14:24:07 +0000 | [diff] [blame] | 636 | // We can remove the instructions now we're no longer iterating through them. |
| 637 | for (auto EV : PrunedInsts) |
| 638 | EV->eraseFromParent(); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 639 | |
Tim Northover | 420a216 | 2014-06-13 14:24:07 +0000 | [diff] [blame] | 640 | if (!CI->use_empty()) { |
| 641 | // Some use of the full struct return that we don't understand has happened, |
| 642 | // so we've got to reconstruct it properly. |
| 643 | Value *Res; |
| 644 | Res = Builder.CreateInsertValue(UndefValue::get(CI->getType()), Loaded, 0); |
| 645 | Res = Builder.CreateInsertValue(Res, Success, 1); |
| 646 | |
| 647 | CI->replaceAllUsesWith(Res); |
| 648 | } |
| 649 | |
| 650 | CI->eraseFromParent(); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 651 | return true; |
| 652 | } |
Robin Morisset | 810739d | 2014-09-25 17:27:43 +0000 | [diff] [blame] | 653 | |
| 654 | bool AtomicExpand::isIdempotentRMW(AtomicRMWInst* RMWI) { |
| 655 | auto C = dyn_cast<ConstantInt>(RMWI->getValOperand()); |
| 656 | if(!C) |
| 657 | return false; |
| 658 | |
| 659 | AtomicRMWInst::BinOp Op = RMWI->getOperation(); |
| 660 | switch(Op) { |
| 661 | case AtomicRMWInst::Add: |
| 662 | case AtomicRMWInst::Sub: |
| 663 | case AtomicRMWInst::Or: |
| 664 | case AtomicRMWInst::Xor: |
| 665 | return C->isZero(); |
| 666 | case AtomicRMWInst::And: |
| 667 | return C->isMinusOne(); |
| 668 | // FIXME: we could also treat Min/Max/UMin/UMax by the INT_MIN/INT_MAX/... |
| 669 | default: |
| 670 | return false; |
| 671 | } |
| 672 | } |
| 673 | |
| 674 | bool AtomicExpand::simplifyIdempotentRMW(AtomicRMWInst* RMWI) { |
Ahmed Bougacha | 49b531a | 2015-09-12 18:51:23 +0000 | [diff] [blame] | 675 | if (auto ResultingLoad = TLI->lowerIdempotentRMWIntoFencedLoad(RMWI)) { |
| 676 | tryExpandAtomicLoad(ResultingLoad); |
| 677 | return true; |
| 678 | } |
Robin Morisset | 810739d | 2014-09-25 17:27:43 +0000 | [diff] [blame] | 679 | return false; |
| 680 | } |
JF Bastien | e8aad29 | 2015-08-03 15:29:47 +0000 | [diff] [blame] | 681 | |
| 682 | bool llvm::expandAtomicRMWToCmpXchg(AtomicRMWInst *AI, |
| 683 | CreateCmpXchgInstFun CreateCmpXchg) { |
| 684 | assert(AI); |
| 685 | |
| 686 | AtomicOrdering MemOpOrder = |
| 687 | AI->getOrdering() == Unordered ? Monotonic : AI->getOrdering(); |
| 688 | Value *Addr = AI->getPointerOperand(); |
| 689 | BasicBlock *BB = AI->getParent(); |
| 690 | Function *F = BB->getParent(); |
| 691 | LLVMContext &Ctx = F->getContext(); |
| 692 | |
| 693 | // Given: atomicrmw some_op iN* %addr, iN %incr ordering |
| 694 | // |
| 695 | // The standard expansion we produce is: |
| 696 | // [...] |
| 697 | // %init_loaded = load atomic iN* %addr |
| 698 | // br label %loop |
| 699 | // loop: |
| 700 | // %loaded = phi iN [ %init_loaded, %entry ], [ %new_loaded, %loop ] |
| 701 | // %new = some_op iN %loaded, %incr |
| 702 | // %pair = cmpxchg iN* %addr, iN %loaded, iN %new |
| 703 | // %new_loaded = extractvalue { iN, i1 } %pair, 0 |
| 704 | // %success = extractvalue { iN, i1 } %pair, 1 |
| 705 | // br i1 %success, label %atomicrmw.end, label %loop |
| 706 | // atomicrmw.end: |
| 707 | // [...] |
Duncan P. N. Exon Smith | 8f11e1a | 2015-10-09 16:54:49 +0000 | [diff] [blame] | 708 | BasicBlock *ExitBB = BB->splitBasicBlock(AI->getIterator(), "atomicrmw.end"); |
JF Bastien | e8aad29 | 2015-08-03 15:29:47 +0000 | [diff] [blame] | 709 | BasicBlock *LoopBB = BasicBlock::Create(Ctx, "atomicrmw.start", F, ExitBB); |
| 710 | |
| 711 | // This grabs the DebugLoc from AI. |
| 712 | IRBuilder<> Builder(AI); |
| 713 | |
| 714 | // The split call above "helpfully" added a branch at the end of BB (to the |
| 715 | // wrong place), but we want a load. It's easiest to just remove |
| 716 | // the branch entirely. |
| 717 | std::prev(BB->end())->eraseFromParent(); |
| 718 | Builder.SetInsertPoint(BB); |
| 719 | LoadInst *InitLoaded = Builder.CreateLoad(Addr); |
| 720 | // Atomics require at least natural alignment. |
Richard Diamond | bd753c9 | 2015-08-06 16:55:03 +0000 | [diff] [blame] | 721 | InitLoaded->setAlignment(AI->getType()->getPrimitiveSizeInBits() / 8); |
JF Bastien | e8aad29 | 2015-08-03 15:29:47 +0000 | [diff] [blame] | 722 | Builder.CreateBr(LoopBB); |
| 723 | |
| 724 | // Start the main loop block now that we've taken care of the preliminaries. |
| 725 | Builder.SetInsertPoint(LoopBB); |
| 726 | PHINode *Loaded = Builder.CreatePHI(AI->getType(), 2, "loaded"); |
| 727 | Loaded->addIncoming(InitLoaded, BB); |
| 728 | |
| 729 | Value *NewVal = |
| 730 | performAtomicOp(AI->getOperation(), Builder, Loaded, AI->getValOperand()); |
| 731 | |
| 732 | Value *NewLoaded = nullptr; |
| 733 | Value *Success = nullptr; |
| 734 | |
| 735 | CreateCmpXchg(Builder, Addr, Loaded, NewVal, MemOpOrder, |
| 736 | Success, NewLoaded); |
| 737 | assert(Success && NewLoaded); |
| 738 | |
| 739 | Loaded->addIncoming(NewLoaded, LoopBB); |
| 740 | |
| 741 | Builder.CreateCondBr(Success, ExitBB, LoopBB); |
| 742 | |
| 743 | Builder.SetInsertPoint(ExitBB, ExitBB->begin()); |
| 744 | |
| 745 | AI->replaceAllUsesWith(NewLoaded); |
| 746 | AI->eraseFromParent(); |
| 747 | |
| 748 | return true; |
| 749 | } |