Tim Northover | 037f26f2 | 2014-04-17 18:22:47 +0000 | [diff] [blame] | 1 | //===-- AtomicExpandLoadLinkedPass.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 |
| 11 | // appropriate (intrinsic-based) ldrex/strex loops. |
| 12 | // |
| 13 | //===----------------------------------------------------------------------===// |
| 14 | |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 15 | #include "llvm/CodeGen/Passes.h" |
| 16 | #include "llvm/IR/Function.h" |
| 17 | #include "llvm/IR/IRBuilder.h" |
| 18 | #include "llvm/IR/Instructions.h" |
| 19 | #include "llvm/IR/Intrinsics.h" |
| 20 | #include "llvm/IR/Module.h" |
| 21 | #include "llvm/Support/Debug.h" |
| 22 | #include "llvm/Target/TargetLowering.h" |
| 23 | #include "llvm/Target/TargetMachine.h" |
| 24 | using namespace llvm; |
| 25 | |
Chandler Carruth | 1b9dde0 | 2014-04-22 02:02:50 +0000 | [diff] [blame] | 26 | #define DEBUG_TYPE "arm-atomic-expand" |
| 27 | |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 28 | namespace { |
Tim Northover | 037f26f2 | 2014-04-17 18:22:47 +0000 | [diff] [blame] | 29 | class AtomicExpandLoadLinked : public FunctionPass { |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 30 | const TargetLowering *TLI; |
| 31 | public: |
| 32 | static char ID; // Pass identification, replacement for typeid |
Craig Topper | 353eda4 | 2014-04-24 06:44:33 +0000 | [diff] [blame] | 33 | explicit AtomicExpandLoadLinked(const TargetMachine *TM = nullptr) |
| 34 | : FunctionPass(ID), TLI(TM ? TM->getTargetLowering() : nullptr) { |
Tim Northover | 037f26f2 | 2014-04-17 18:22:47 +0000 | [diff] [blame] | 35 | initializeAtomicExpandLoadLinkedPass(*PassRegistry::getPassRegistry()); |
| 36 | } |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 37 | |
| 38 | bool runOnFunction(Function &F) override; |
| 39 | bool expandAtomicInsts(Function &F); |
| 40 | |
| 41 | bool expandAtomicLoad(LoadInst *LI); |
| 42 | bool expandAtomicStore(StoreInst *LI); |
| 43 | bool expandAtomicRMW(AtomicRMWInst *AI); |
| 44 | bool expandAtomicCmpXchg(AtomicCmpXchgInst *CI); |
| 45 | |
| 46 | AtomicOrdering insertLeadingFence(IRBuilder<> &Builder, AtomicOrdering Ord); |
| 47 | void insertTrailingFence(IRBuilder<> &Builder, AtomicOrdering Ord); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 48 | }; |
| 49 | } |
| 50 | |
Tim Northover | 037f26f2 | 2014-04-17 18:22:47 +0000 | [diff] [blame] | 51 | char AtomicExpandLoadLinked::ID = 0; |
| 52 | char &llvm::AtomicExpandLoadLinkedID = AtomicExpandLoadLinked::ID; |
Jiangning Liu | d623c52 | 2014-06-11 07:04:37 +0000 | [diff] [blame] | 53 | INITIALIZE_TM_PASS(AtomicExpandLoadLinked, "atomic-ll-sc", |
| 54 | "Expand Atomic calls in terms of load-linked & store-conditional", |
| 55 | false, false) |
Tim Northover | 037f26f2 | 2014-04-17 18:22:47 +0000 | [diff] [blame] | 56 | |
| 57 | FunctionPass *llvm::createAtomicExpandLoadLinkedPass(const TargetMachine *TM) { |
| 58 | return new AtomicExpandLoadLinked(TM); |
| 59 | } |
| 60 | |
| 61 | bool AtomicExpandLoadLinked::runOnFunction(Function &F) { |
| 62 | if (!TLI) |
| 63 | return false; |
| 64 | |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 65 | SmallVector<Instruction *, 1> AtomicInsts; |
| 66 | |
| 67 | // Changing control-flow while iterating through it is a bad idea, so gather a |
| 68 | // list of all atomic instructions before we start. |
| 69 | for (BasicBlock &BB : F) |
| 70 | for (Instruction &Inst : BB) { |
| 71 | if (isa<AtomicRMWInst>(&Inst) || isa<AtomicCmpXchgInst>(&Inst) || |
| 72 | (isa<LoadInst>(&Inst) && cast<LoadInst>(&Inst)->isAtomic()) || |
| 73 | (isa<StoreInst>(&Inst) && cast<StoreInst>(&Inst)->isAtomic())) |
| 74 | AtomicInsts.push_back(&Inst); |
| 75 | } |
| 76 | |
| 77 | bool MadeChange = false; |
| 78 | for (Instruction *Inst : AtomicInsts) { |
Tim Northover | 037f26f2 | 2014-04-17 18:22:47 +0000 | [diff] [blame] | 79 | if (!TLI->shouldExpandAtomicInIR(Inst)) |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 80 | continue; |
| 81 | |
| 82 | if (AtomicRMWInst *AI = dyn_cast<AtomicRMWInst>(Inst)) |
| 83 | MadeChange |= expandAtomicRMW(AI); |
| 84 | else if (AtomicCmpXchgInst *CI = dyn_cast<AtomicCmpXchgInst>(Inst)) |
| 85 | MadeChange |= expandAtomicCmpXchg(CI); |
| 86 | else if (LoadInst *LI = dyn_cast<LoadInst>(Inst)) |
| 87 | MadeChange |= expandAtomicLoad(LI); |
| 88 | else if (StoreInst *SI = dyn_cast<StoreInst>(Inst)) |
| 89 | MadeChange |= expandAtomicStore(SI); |
| 90 | else |
| 91 | llvm_unreachable("Unknown atomic instruction"); |
| 92 | } |
| 93 | |
| 94 | return MadeChange; |
| 95 | } |
| 96 | |
Tim Northover | 037f26f2 | 2014-04-17 18:22:47 +0000 | [diff] [blame] | 97 | bool AtomicExpandLoadLinked::expandAtomicLoad(LoadInst *LI) { |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 98 | // Load instructions don't actually need a leading fence, even in the |
| 99 | // SequentiallyConsistent case. |
| 100 | AtomicOrdering MemOpOrder = |
| 101 | TLI->getInsertFencesForAtomic() ? Monotonic : LI->getOrdering(); |
| 102 | |
| 103 | // The only 64-bit load guaranteed to be single-copy atomic by the ARM ARM is |
| 104 | // an ldrexd (A3.5.3). |
| 105 | IRBuilder<> Builder(LI); |
Tim Northover | 037f26f2 | 2014-04-17 18:22:47 +0000 | [diff] [blame] | 106 | Value *Val = |
| 107 | TLI->emitLoadLinked(Builder, LI->getPointerOperand(), MemOpOrder); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 108 | |
| 109 | insertTrailingFence(Builder, LI->getOrdering()); |
| 110 | |
| 111 | LI->replaceAllUsesWith(Val); |
| 112 | LI->eraseFromParent(); |
| 113 | |
| 114 | return true; |
| 115 | } |
| 116 | |
Tim Northover | 037f26f2 | 2014-04-17 18:22:47 +0000 | [diff] [blame] | 117 | bool AtomicExpandLoadLinked::expandAtomicStore(StoreInst *SI) { |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 118 | // The only atomic 64-bit store on ARM is an strexd that succeeds, which means |
| 119 | // we need a loop and the entire instruction is essentially an "atomicrmw |
| 120 | // xchg" that ignores the value loaded. |
| 121 | IRBuilder<> Builder(SI); |
| 122 | AtomicRMWInst *AI = |
| 123 | Builder.CreateAtomicRMW(AtomicRMWInst::Xchg, SI->getPointerOperand(), |
| 124 | SI->getValueOperand(), SI->getOrdering()); |
| 125 | SI->eraseFromParent(); |
| 126 | |
| 127 | // Now we have an appropriate swap instruction, lower it as usual. |
| 128 | return expandAtomicRMW(AI); |
| 129 | } |
| 130 | |
Tim Northover | 037f26f2 | 2014-04-17 18:22:47 +0000 | [diff] [blame] | 131 | bool AtomicExpandLoadLinked::expandAtomicRMW(AtomicRMWInst *AI) { |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 132 | AtomicOrdering Order = AI->getOrdering(); |
| 133 | Value *Addr = AI->getPointerOperand(); |
| 134 | BasicBlock *BB = AI->getParent(); |
| 135 | Function *F = BB->getParent(); |
| 136 | LLVMContext &Ctx = F->getContext(); |
| 137 | |
| 138 | // Given: atomicrmw some_op iN* %addr, iN %incr ordering |
| 139 | // |
| 140 | // The standard expansion we produce is: |
| 141 | // [...] |
| 142 | // fence? |
| 143 | // atomicrmw.start: |
| 144 | // %loaded = @load.linked(%addr) |
| 145 | // %new = some_op iN %loaded, %incr |
| 146 | // %stored = @store_conditional(%new, %addr) |
| 147 | // %try_again = icmp i32 ne %stored, 0 |
| 148 | // br i1 %try_again, label %loop, label %atomicrmw.end |
| 149 | // atomicrmw.end: |
| 150 | // fence? |
| 151 | // [...] |
| 152 | BasicBlock *ExitBB = BB->splitBasicBlock(AI, "atomicrmw.end"); |
| 153 | BasicBlock *LoopBB = BasicBlock::Create(Ctx, "atomicrmw.start", F, ExitBB); |
| 154 | |
| 155 | // This grabs the DebugLoc from AI. |
| 156 | IRBuilder<> Builder(AI); |
| 157 | |
| 158 | // The split call above "helpfully" added a branch at the end of BB (to the |
| 159 | // wrong place), but we might want a fence too. It's easiest to just remove |
| 160 | // the branch entirely. |
| 161 | std::prev(BB->end())->eraseFromParent(); |
| 162 | Builder.SetInsertPoint(BB); |
| 163 | AtomicOrdering MemOpOrder = insertLeadingFence(Builder, Order); |
| 164 | Builder.CreateBr(LoopBB); |
| 165 | |
| 166 | // Start the main loop block now that we've taken care of the preliminaries. |
| 167 | Builder.SetInsertPoint(LoopBB); |
Tim Northover | 037f26f2 | 2014-04-17 18:22:47 +0000 | [diff] [blame] | 168 | Value *Loaded = TLI->emitLoadLinked(Builder, Addr, MemOpOrder); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 169 | |
| 170 | Value *NewVal; |
| 171 | switch (AI->getOperation()) { |
| 172 | case AtomicRMWInst::Xchg: |
| 173 | NewVal = AI->getValOperand(); |
| 174 | break; |
| 175 | case AtomicRMWInst::Add: |
| 176 | NewVal = Builder.CreateAdd(Loaded, AI->getValOperand(), "new"); |
| 177 | break; |
| 178 | case AtomicRMWInst::Sub: |
| 179 | NewVal = Builder.CreateSub(Loaded, AI->getValOperand(), "new"); |
| 180 | break; |
| 181 | case AtomicRMWInst::And: |
| 182 | NewVal = Builder.CreateAnd(Loaded, AI->getValOperand(), "new"); |
| 183 | break; |
| 184 | case AtomicRMWInst::Nand: |
| 185 | NewVal = Builder.CreateAnd(Loaded, Builder.CreateNot(AI->getValOperand()), |
| 186 | "new"); |
| 187 | break; |
| 188 | case AtomicRMWInst::Or: |
| 189 | NewVal = Builder.CreateOr(Loaded, AI->getValOperand(), "new"); |
| 190 | break; |
| 191 | case AtomicRMWInst::Xor: |
| 192 | NewVal = Builder.CreateXor(Loaded, AI->getValOperand(), "new"); |
| 193 | break; |
| 194 | case AtomicRMWInst::Max: |
| 195 | NewVal = Builder.CreateICmpSGT(Loaded, AI->getValOperand()); |
| 196 | NewVal = Builder.CreateSelect(NewVal, Loaded, AI->getValOperand(), "new"); |
| 197 | break; |
| 198 | case AtomicRMWInst::Min: |
| 199 | NewVal = Builder.CreateICmpSLE(Loaded, AI->getValOperand()); |
| 200 | NewVal = Builder.CreateSelect(NewVal, Loaded, AI->getValOperand(), "new"); |
| 201 | break; |
| 202 | case AtomicRMWInst::UMax: |
| 203 | NewVal = Builder.CreateICmpUGT(Loaded, AI->getValOperand()); |
| 204 | NewVal = Builder.CreateSelect(NewVal, Loaded, AI->getValOperand(), "new"); |
| 205 | break; |
| 206 | case AtomicRMWInst::UMin: |
| 207 | NewVal = Builder.CreateICmpULE(Loaded, AI->getValOperand()); |
| 208 | NewVal = Builder.CreateSelect(NewVal, Loaded, AI->getValOperand(), "new"); |
| 209 | break; |
| 210 | default: |
| 211 | llvm_unreachable("Unknown atomic op"); |
| 212 | } |
| 213 | |
Tim Northover | 037f26f2 | 2014-04-17 18:22:47 +0000 | [diff] [blame] | 214 | Value *StoreSuccess = |
| 215 | TLI->emitStoreConditional(Builder, NewVal, Addr, MemOpOrder); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 216 | Value *TryAgain = Builder.CreateICmpNE( |
| 217 | StoreSuccess, ConstantInt::get(IntegerType::get(Ctx, 32), 0), "tryagain"); |
| 218 | Builder.CreateCondBr(TryAgain, LoopBB, ExitBB); |
| 219 | |
| 220 | Builder.SetInsertPoint(ExitBB, ExitBB->begin()); |
| 221 | insertTrailingFence(Builder, Order); |
| 222 | |
| 223 | AI->replaceAllUsesWith(Loaded); |
| 224 | AI->eraseFromParent(); |
| 225 | |
| 226 | return true; |
| 227 | } |
| 228 | |
Tim Northover | 037f26f2 | 2014-04-17 18:22:47 +0000 | [diff] [blame] | 229 | bool AtomicExpandLoadLinked::expandAtomicCmpXchg(AtomicCmpXchgInst *CI) { |
Tim Northover | 70450c5 | 2014-04-03 13:06:54 +0000 | [diff] [blame] | 230 | AtomicOrdering SuccessOrder = CI->getSuccessOrdering(); |
| 231 | AtomicOrdering FailureOrder = CI->getFailureOrdering(); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 232 | Value *Addr = CI->getPointerOperand(); |
| 233 | BasicBlock *BB = CI->getParent(); |
| 234 | Function *F = BB->getParent(); |
| 235 | LLVMContext &Ctx = F->getContext(); |
| 236 | |
| 237 | // Given: cmpxchg some_op iN* %addr, iN %desired, iN %new success_ord fail_ord |
| 238 | // |
Tim Northover | 70450c5 | 2014-04-03 13:06:54 +0000 | [diff] [blame] | 239 | // The full expansion we produce is: |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 240 | // [...] |
| 241 | // fence? |
| 242 | // cmpxchg.start: |
| 243 | // %loaded = @load.linked(%addr) |
| 244 | // %should_store = icmp eq %loaded, %desired |
Tim Northover | 70450c5 | 2014-04-03 13:06:54 +0000 | [diff] [blame] | 245 | // br i1 %should_store, label %cmpxchg.trystore, |
| 246 | // label %cmpxchg.end/%cmpxchg.barrier |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 247 | // cmpxchg.trystore: |
| 248 | // %stored = @store_conditional(%new, %addr) |
| 249 | // %try_again = icmp i32 ne %stored, 0 |
| 250 | // br i1 %try_again, label %loop, label %cmpxchg.end |
Tim Northover | 70450c5 | 2014-04-03 13:06:54 +0000 | [diff] [blame] | 251 | // cmpxchg.barrier: |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 252 | // fence? |
Tim Northover | 70450c5 | 2014-04-03 13:06:54 +0000 | [diff] [blame] | 253 | // br label %cmpxchg.end |
| 254 | // cmpxchg.end: |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 255 | // [...] |
| 256 | BasicBlock *ExitBB = BB->splitBasicBlock(CI, "cmpxchg.end"); |
Tim Northover | 037f26f2 | 2014-04-17 18:22:47 +0000 | [diff] [blame] | 257 | auto BarrierBB = BasicBlock::Create(Ctx, "cmpxchg.barrier", F, ExitBB); |
| 258 | auto TryStoreBB = BasicBlock::Create(Ctx, "cmpxchg.trystore", F, BarrierBB); |
Tim Northover | 70450c5 | 2014-04-03 13:06:54 +0000 | [diff] [blame] | 259 | auto LoopBB = BasicBlock::Create(Ctx, "cmpxchg.start", F, TryStoreBB); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 260 | |
| 261 | // This grabs the DebugLoc from CI |
| 262 | IRBuilder<> Builder(CI); |
| 263 | |
| 264 | // The split call above "helpfully" added a branch at the end of BB (to the |
| 265 | // wrong place), but we might want a fence too. It's easiest to just remove |
| 266 | // the branch entirely. |
| 267 | std::prev(BB->end())->eraseFromParent(); |
| 268 | Builder.SetInsertPoint(BB); |
Tim Northover | 70450c5 | 2014-04-03 13:06:54 +0000 | [diff] [blame] | 269 | AtomicOrdering MemOpOrder = insertLeadingFence(Builder, SuccessOrder); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 270 | Builder.CreateBr(LoopBB); |
| 271 | |
| 272 | // Start the main loop block now that we've taken care of the preliminaries. |
| 273 | Builder.SetInsertPoint(LoopBB); |
Tim Northover | 037f26f2 | 2014-04-17 18:22:47 +0000 | [diff] [blame] | 274 | Value *Loaded = TLI->emitLoadLinked(Builder, Addr, MemOpOrder); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 275 | Value *ShouldStore = |
| 276 | Builder.CreateICmpEQ(Loaded, CI->getCompareOperand(), "should_store"); |
Tim Northover | 70450c5 | 2014-04-03 13:06:54 +0000 | [diff] [blame] | 277 | |
| 278 | // If the the cmpxchg doesn't actually need any ordering when it fails, we can |
| 279 | // jump straight past that fence instruction (if it exists). |
| 280 | BasicBlock *FailureBB = FailureOrder == Monotonic ? ExitBB : BarrierBB; |
| 281 | Builder.CreateCondBr(ShouldStore, TryStoreBB, FailureBB); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 282 | |
| 283 | Builder.SetInsertPoint(TryStoreBB); |
Tim Northover | 037f26f2 | 2014-04-17 18:22:47 +0000 | [diff] [blame] | 284 | Value *StoreSuccess = TLI->emitStoreConditional( |
| 285 | Builder, CI->getNewValOperand(), Addr, MemOpOrder); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 286 | Value *TryAgain = Builder.CreateICmpNE( |
| 287 | StoreSuccess, ConstantInt::get(Type::getInt32Ty(Ctx), 0), "success"); |
Tim Northover | 70450c5 | 2014-04-03 13:06:54 +0000 | [diff] [blame] | 288 | Builder.CreateCondBr(TryAgain, LoopBB, BarrierBB); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 289 | |
Tim Northover | b4ddc08 | 2014-05-30 10:09:59 +0000 | [diff] [blame] | 290 | // Make sure later instructions don't get reordered with a fence if necessary. |
Tim Northover | 70450c5 | 2014-04-03 13:06:54 +0000 | [diff] [blame] | 291 | Builder.SetInsertPoint(BarrierBB); |
| 292 | insertTrailingFence(Builder, SuccessOrder); |
| 293 | Builder.CreateBr(ExitBB); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 294 | |
Tim Northover | b4ddc08 | 2014-05-30 10:09:59 +0000 | [diff] [blame] | 295 | // Finally, we have control-flow based knowledge of whether the cmpxchg |
| 296 | // succeeded or not. We expose this to later passes by converting any |
| 297 | // subsequent "icmp eq/ne %loaded, %oldval" into a use of an appropriate PHI. |
| 298 | |
| 299 | // Setup the builder so we can create any PHIs we need. |
| 300 | Builder.SetInsertPoint(FailureBB, FailureBB->begin()); |
| 301 | BasicBlock *SuccessBB = FailureOrder == Monotonic ? BarrierBB : TryStoreBB; |
Tim Northover | 420a216 | 2014-06-13 14:24:07 +0000 | [diff] [blame^] | 302 | PHINode *Success = Builder.CreatePHI(Type::getInt1Ty(Ctx), 2); |
| 303 | Success->addIncoming(ConstantInt::getTrue(Ctx), SuccessBB); |
| 304 | Success->addIncoming(ConstantInt::getFalse(Ctx), LoopBB); |
Tim Northover | b4ddc08 | 2014-05-30 10:09:59 +0000 | [diff] [blame] | 305 | |
| 306 | // Look for any users of the cmpxchg that are just comparing the loaded value |
| 307 | // against the desired one, and replace them with the CFG-derived version. |
Tim Northover | 420a216 | 2014-06-13 14:24:07 +0000 | [diff] [blame^] | 308 | SmallVector<ExtractValueInst *, 2> PrunedInsts; |
Tim Northover | b4ddc08 | 2014-05-30 10:09:59 +0000 | [diff] [blame] | 309 | for (auto User : CI->users()) { |
Tim Northover | 420a216 | 2014-06-13 14:24:07 +0000 | [diff] [blame^] | 310 | ExtractValueInst *EV = dyn_cast<ExtractValueInst>(User); |
| 311 | if (!EV) |
Tim Northover | b4ddc08 | 2014-05-30 10:09:59 +0000 | [diff] [blame] | 312 | continue; |
| 313 | |
Tim Northover | 420a216 | 2014-06-13 14:24:07 +0000 | [diff] [blame^] | 314 | assert(EV->getNumIndices() == 1 && EV->getIndices()[0] <= 1 && |
| 315 | "weird extraction from { iN, i1 }"); |
Tim Northover | b4ddc08 | 2014-05-30 10:09:59 +0000 | [diff] [blame] | 316 | |
Tim Northover | 420a216 | 2014-06-13 14:24:07 +0000 | [diff] [blame^] | 317 | if (EV->getIndices()[0] == 0) |
| 318 | EV->replaceAllUsesWith(Loaded); |
| 319 | else |
| 320 | EV->replaceAllUsesWith(Success); |
| 321 | |
| 322 | PrunedInsts.push_back(EV); |
Tim Northover | b4ddc08 | 2014-05-30 10:09:59 +0000 | [diff] [blame] | 323 | } |
| 324 | |
Tim Northover | 420a216 | 2014-06-13 14:24:07 +0000 | [diff] [blame^] | 325 | // We can remove the instructions now we're no longer iterating through them. |
| 326 | for (auto EV : PrunedInsts) |
| 327 | EV->eraseFromParent(); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 328 | |
Tim Northover | 420a216 | 2014-06-13 14:24:07 +0000 | [diff] [blame^] | 329 | if (!CI->use_empty()) { |
| 330 | // Some use of the full struct return that we don't understand has happened, |
| 331 | // so we've got to reconstruct it properly. |
| 332 | Value *Res; |
| 333 | Res = Builder.CreateInsertValue(UndefValue::get(CI->getType()), Loaded, 0); |
| 334 | Res = Builder.CreateInsertValue(Res, Success, 1); |
| 335 | |
| 336 | CI->replaceAllUsesWith(Res); |
| 337 | } |
| 338 | |
| 339 | CI->eraseFromParent(); |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 340 | return true; |
| 341 | } |
| 342 | |
Tim Northover | 037f26f2 | 2014-04-17 18:22:47 +0000 | [diff] [blame] | 343 | AtomicOrdering AtomicExpandLoadLinked::insertLeadingFence(IRBuilder<> &Builder, |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 344 | AtomicOrdering Ord) { |
| 345 | if (!TLI->getInsertFencesForAtomic()) |
| 346 | return Ord; |
| 347 | |
| 348 | if (Ord == Release || Ord == AcquireRelease || Ord == SequentiallyConsistent) |
| 349 | Builder.CreateFence(Release); |
| 350 | |
| 351 | // The exclusive operations don't need any barrier if we're adding separate |
| 352 | // fences. |
| 353 | return Monotonic; |
| 354 | } |
| 355 | |
Tim Northover | 037f26f2 | 2014-04-17 18:22:47 +0000 | [diff] [blame] | 356 | void AtomicExpandLoadLinked::insertTrailingFence(IRBuilder<> &Builder, |
Tim Northover | c882eb0 | 2014-04-03 11:44:58 +0000 | [diff] [blame] | 357 | AtomicOrdering Ord) { |
| 358 | if (!TLI->getInsertFencesForAtomic()) |
| 359 | return; |
| 360 | |
| 361 | if (Ord == Acquire || Ord == AcquireRelease) |
| 362 | Builder.CreateFence(Acquire); |
| 363 | else if (Ord == SequentiallyConsistent) |
| 364 | Builder.CreateFence(SequentiallyConsistent); |
| 365 | } |