Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 1 | //===-- StatepointLowering.cpp - SDAGBuilder's statepoint code -----------===// |
| 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 includes support code use by SelectionDAGBuilder when lowering a |
| 11 | // statepoint sequence in SelectionDAG IR. |
| 12 | // |
| 13 | //===----------------------------------------------------------------------===// |
| 14 | |
| 15 | #include "StatepointLowering.h" |
| 16 | #include "SelectionDAGBuilder.h" |
| 17 | #include "llvm/ADT/SmallSet.h" |
| 18 | #include "llvm/ADT/Statistic.h" |
| 19 | #include "llvm/CodeGen/FunctionLoweringInfo.h" |
Philip Reames | cb0f947 | 2015-12-23 23:44:28 +0000 | [diff] [blame] | 20 | #include "llvm/CodeGen/MachineFrameInfo.h" |
Chandler Carruth | 71f308a | 2015-02-13 09:09:03 +0000 | [diff] [blame] | 21 | #include "llvm/CodeGen/GCMetadata.h" |
Philip Reames | 56a0393 | 2015-01-26 18:26:35 +0000 | [diff] [blame] | 22 | #include "llvm/CodeGen/GCStrategy.h" |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 23 | #include "llvm/CodeGen/SelectionDAG.h" |
| 24 | #include "llvm/CodeGen/StackMaps.h" |
| 25 | #include "llvm/IR/CallingConv.h" |
| 26 | #include "llvm/IR/Instructions.h" |
| 27 | #include "llvm/IR/IntrinsicInst.h" |
| 28 | #include "llvm/IR/Intrinsics.h" |
| 29 | #include "llvm/IR/Statepoint.h" |
| 30 | #include "llvm/Target/TargetLowering.h" |
| 31 | #include <algorithm> |
| 32 | using namespace llvm; |
| 33 | |
| 34 | #define DEBUG_TYPE "statepoint-lowering" |
| 35 | |
| 36 | STATISTIC(NumSlotsAllocatedForStatepoints, |
| 37 | "Number of stack slots allocated for statepoints"); |
| 38 | STATISTIC(NumOfStatepoints, "Number of statepoint nodes encountered"); |
| 39 | STATISTIC(StatepointMaxSlotsRequired, |
| 40 | "Maximum number of stack slots required for a singe statepoint"); |
| 41 | |
Pat Gavlin | c7dc6d6ee | 2015-05-12 19:50:19 +0000 | [diff] [blame] | 42 | static void pushStackMapConstant(SmallVectorImpl<SDValue>& Ops, |
| 43 | SelectionDAGBuilder &Builder, uint64_t Value) { |
| 44 | SDLoc L = Builder.getCurSDLoc(); |
| 45 | Ops.push_back(Builder.DAG.getTargetConstant(StackMaps::ConstantOp, L, |
| 46 | MVT::i64)); |
| 47 | Ops.push_back(Builder.DAG.getTargetConstant(Value, L, MVT::i64)); |
| 48 | } |
| 49 | |
Pat Gavlin | 022c5ac | 2015-04-29 21:52:45 +0000 | [diff] [blame] | 50 | void StatepointLoweringState::startNewStatepoint(SelectionDAGBuilder &Builder) { |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 51 | // Consistency check |
| 52 | assert(PendingGCRelocateCalls.empty() && |
| 53 | "Trying to visit statepoint before finished processing previous one"); |
| 54 | Locations.clear(); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 55 | NextSlotToAllocate = 0; |
| 56 | // Need to resize this on each safepoint - we need the two to stay in |
| 57 | // sync and the clear patterns of a SelectionDAGBuilder have no relation |
| 58 | // to FunctionLoweringInfo. |
| 59 | AllocatedStackSlots.resize(Builder.FuncInfo.StatepointStackSlots.size()); |
| 60 | for (size_t i = 0; i < AllocatedStackSlots.size(); i++) { |
| 61 | AllocatedStackSlots[i] = false; |
| 62 | } |
| 63 | } |
Igor Laevsky | 423bc9ec | 2015-05-20 11:37:25 +0000 | [diff] [blame] | 64 | |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 65 | void StatepointLoweringState::clear() { |
| 66 | Locations.clear(); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 67 | AllocatedStackSlots.clear(); |
| 68 | assert(PendingGCRelocateCalls.empty() && |
| 69 | "cleared before statepoint sequence completed"); |
| 70 | } |
| 71 | |
| 72 | SDValue |
| 73 | StatepointLoweringState::allocateStackSlot(EVT ValueType, |
| 74 | SelectionDAGBuilder &Builder) { |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 75 | NumSlotsAllocatedForStatepoints++; |
| 76 | |
Sanjoy Das | 7b2e91f | 2016-02-19 17:15:17 +0000 | [diff] [blame^] | 77 | // First look for a previously created stack slot which is not in |
| 78 | // use (accounting for the fact arbitrary slots may already be |
| 79 | // reserved), or to create a new stack slot and use it. |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 80 | |
Sanjoy Das | 7b2e91f | 2016-02-19 17:15:17 +0000 | [diff] [blame^] | 81 | const size_t NumSlots = AllocatedStackSlots.size(); |
| 82 | assert(NextSlotToAllocate <= NumSlots && "Broken invariant"); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 83 | |
Sanjoy Das | 7b2e91f | 2016-02-19 17:15:17 +0000 | [diff] [blame^] | 84 | for (; NextSlotToAllocate < NumSlots; NextSlotToAllocate++) { |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 85 | if (!AllocatedStackSlots[NextSlotToAllocate]) { |
| 86 | const int FI = Builder.FuncInfo.StatepointStackSlots[NextSlotToAllocate]; |
| 87 | AllocatedStackSlots[NextSlotToAllocate] = true; |
| 88 | return Builder.DAG.getFrameIndex(FI, ValueType); |
| 89 | } |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 90 | } |
Sanjoy Das | 7b2e91f | 2016-02-19 17:15:17 +0000 | [diff] [blame^] | 91 | |
| 92 | // Couldn't find a free slot, so create a new one: |
| 93 | |
| 94 | StatepointMaxSlotsRequired = |
| 95 | std::max<unsigned long>(StatepointMaxSlotsRequired, NumSlots + 1); |
| 96 | |
| 97 | SDValue SpillSlot = Builder.DAG.CreateStackTemporary(ValueType); |
| 98 | const unsigned FI = cast<FrameIndexSDNode>(SpillSlot)->getIndex(); |
| 99 | auto *MFI = Builder.DAG.getMachineFunction().getFrameInfo(); |
| 100 | MFI->markAsStatepointSpillSlotObjectIndex(FI); |
| 101 | |
| 102 | Builder.FuncInfo.StatepointStackSlots.push_back(FI); |
| 103 | AllocatedStackSlots.push_back(true); |
| 104 | return SpillSlot; |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 105 | } |
| 106 | |
Igor Laevsky | 346ff62 | 2015-06-10 12:31:53 +0000 | [diff] [blame] | 107 | /// Utility function for reservePreviousStackSlotForValue. Tries to find |
| 108 | /// stack slot index to which we have spilled value for previous statepoints. |
| 109 | /// LookUpDepth specifies maximum DFS depth this function is allowed to look. |
| 110 | static Optional<int> findPreviousSpillSlot(const Value *Val, |
| 111 | SelectionDAGBuilder &Builder, |
| 112 | int LookUpDepth) { |
Benjamin Kramer | df005cb | 2015-08-08 18:27:36 +0000 | [diff] [blame] | 113 | // Can not look any further - give up now |
Igor Laevsky | 346ff62 | 2015-06-10 12:31:53 +0000 | [diff] [blame] | 114 | if (LookUpDepth <= 0) |
Sanjoy Das | 86d7d83 | 2016-02-05 23:40:04 +0000 | [diff] [blame] | 115 | return None; |
Igor Laevsky | 346ff62 | 2015-06-10 12:31:53 +0000 | [diff] [blame] | 116 | |
| 117 | // Spill location is known for gc relocates |
Manuel Jacob | 83eefa6 | 2016-01-05 04:03:00 +0000 | [diff] [blame] | 118 | if (const auto *Relocate = dyn_cast<GCRelocateInst>(Val)) { |
Igor Laevsky | 346ff62 | 2015-06-10 12:31:53 +0000 | [diff] [blame] | 119 | FunctionLoweringInfo::StatepointSpilledValueMapTy &SpillMap = |
Manuel Jacob | 83eefa6 | 2016-01-05 04:03:00 +0000 | [diff] [blame] | 120 | Builder.FuncInfo.StatepointRelocatedValues[Relocate->getStatepoint()]; |
Igor Laevsky | 346ff62 | 2015-06-10 12:31:53 +0000 | [diff] [blame] | 121 | |
Manuel Jacob | 83eefa6 | 2016-01-05 04:03:00 +0000 | [diff] [blame] | 122 | auto It = SpillMap.find(Relocate->getDerivedPtr()); |
Igor Laevsky | 346ff62 | 2015-06-10 12:31:53 +0000 | [diff] [blame] | 123 | if (It == SpillMap.end()) |
Sanjoy Das | 86d7d83 | 2016-02-05 23:40:04 +0000 | [diff] [blame] | 124 | return None; |
Igor Laevsky | 346ff62 | 2015-06-10 12:31:53 +0000 | [diff] [blame] | 125 | |
| 126 | return It->second; |
| 127 | } |
| 128 | |
| 129 | // Look through bitcast instructions. |
| 130 | if (const BitCastInst *Cast = dyn_cast<BitCastInst>(Val)) { |
| 131 | return findPreviousSpillSlot(Cast->getOperand(0), Builder, LookUpDepth - 1); |
| 132 | } |
| 133 | |
| 134 | // Look through phi nodes |
| 135 | // All incoming values should have same known stack slot, otherwise result |
| 136 | // is unknown. |
| 137 | if (const PHINode *Phi = dyn_cast<PHINode>(Val)) { |
| 138 | Optional<int> MergedResult = None; |
| 139 | |
| 140 | for (auto &IncomingValue : Phi->incoming_values()) { |
| 141 | Optional<int> SpillSlot = |
| 142 | findPreviousSpillSlot(IncomingValue, Builder, LookUpDepth - 1); |
| 143 | if (!SpillSlot.hasValue()) |
Sanjoy Das | 86d7d83 | 2016-02-05 23:40:04 +0000 | [diff] [blame] | 144 | return None; |
Igor Laevsky | 346ff62 | 2015-06-10 12:31:53 +0000 | [diff] [blame] | 145 | |
| 146 | if (MergedResult.hasValue() && *MergedResult != *SpillSlot) |
Sanjoy Das | 86d7d83 | 2016-02-05 23:40:04 +0000 | [diff] [blame] | 147 | return None; |
Igor Laevsky | 346ff62 | 2015-06-10 12:31:53 +0000 | [diff] [blame] | 148 | |
| 149 | MergedResult = SpillSlot; |
| 150 | } |
| 151 | return MergedResult; |
| 152 | } |
| 153 | |
| 154 | // TODO: We can do better for PHI nodes. In cases like this: |
| 155 | // ptr = phi(relocated_pointer, not_relocated_pointer) |
| 156 | // statepoint(ptr) |
| 157 | // We will return that stack slot for ptr is unknown. And later we might |
| 158 | // assign different stack slots for ptr and relocated_pointer. This limits |
| 159 | // llvm's ability to remove redundant stores. |
| 160 | // Unfortunately it's hard to accomplish in current infrastructure. |
| 161 | // We use this function to eliminate spill store completely, while |
| 162 | // in example we still need to emit store, but instead of any location |
| 163 | // we need to use special "preferred" location. |
| 164 | |
| 165 | // TODO: handle simple updates. If a value is modified and the original |
| 166 | // value is no longer live, it would be nice to put the modified value in the |
| 167 | // same slot. This allows folding of the memory accesses for some |
| 168 | // instructions types (like an increment). |
| 169 | // statepoint (i) |
| 170 | // i1 = i+1 |
| 171 | // statepoint (i1) |
| 172 | // However we need to be careful for cases like this: |
| 173 | // statepoint(i) |
| 174 | // i1 = i+1 |
| 175 | // statepoint(i, i1) |
| 176 | // Here we want to reserve spill slot for 'i', but not for 'i+1'. If we just |
| 177 | // put handling of simple modifications in this function like it's done |
| 178 | // for bitcasts we might end up reserving i's slot for 'i+1' because order in |
| 179 | // which we visit values is unspecified. |
| 180 | |
| 181 | // Don't know any information about this instruction |
Sanjoy Das | 86d7d83 | 2016-02-05 23:40:04 +0000 | [diff] [blame] | 182 | return None; |
Igor Laevsky | 346ff62 | 2015-06-10 12:31:53 +0000 | [diff] [blame] | 183 | } |
| 184 | |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 185 | /// Try to find existing copies of the incoming values in stack slots used for |
| 186 | /// statepoint spilling. If we can find a spill slot for the incoming value, |
| 187 | /// mark that slot as allocated, and reuse the same slot for this safepoint. |
Benjamin Kramer | df005cb | 2015-08-08 18:27:36 +0000 | [diff] [blame] | 188 | /// This helps to avoid series of loads and stores that only serve to reshuffle |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 189 | /// values on the stack between calls. |
Igor Laevsky | 346ff62 | 2015-06-10 12:31:53 +0000 | [diff] [blame] | 190 | static void reservePreviousStackSlotForValue(const Value *IncomingValue, |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 191 | SelectionDAGBuilder &Builder) { |
| 192 | |
Igor Laevsky | 346ff62 | 2015-06-10 12:31:53 +0000 | [diff] [blame] | 193 | SDValue Incoming = Builder.getValue(IncomingValue); |
| 194 | |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 195 | if (isa<ConstantSDNode>(Incoming) || isa<FrameIndexSDNode>(Incoming)) { |
| 196 | // We won't need to spill this, so no need to check for previously |
| 197 | // allocated stack slots |
| 198 | return; |
| 199 | } |
| 200 | |
Igor Laevsky | 346ff62 | 2015-06-10 12:31:53 +0000 | [diff] [blame] | 201 | SDValue OldLocation = Builder.StatepointLowering.getLocation(Incoming); |
| 202 | if (OldLocation.getNode()) |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 203 | // duplicates in input |
| 204 | return; |
Igor Laevsky | 346ff62 | 2015-06-10 12:31:53 +0000 | [diff] [blame] | 205 | |
| 206 | const int LookUpDepth = 6; |
| 207 | Optional<int> Index = |
| 208 | findPreviousSpillSlot(IncomingValue, Builder, LookUpDepth); |
| 209 | if (!Index.hasValue()) |
| 210 | return; |
| 211 | |
| 212 | auto Itr = std::find(Builder.FuncInfo.StatepointStackSlots.begin(), |
| 213 | Builder.FuncInfo.StatepointStackSlots.end(), *Index); |
| 214 | assert(Itr != Builder.FuncInfo.StatepointStackSlots.end() && |
| 215 | "value spilled to the unknown stack slot"); |
| 216 | |
| 217 | // This is one of our dedicated lowering slots |
| 218 | const int Offset = |
| 219 | std::distance(Builder.FuncInfo.StatepointStackSlots.begin(), Itr); |
| 220 | if (Builder.StatepointLowering.isStackSlotAllocated(Offset)) { |
| 221 | // stack slot already assigned to someone else, can't use it! |
| 222 | // TODO: currently we reserve space for gc arguments after doing |
| 223 | // normal allocation for deopt arguments. We should reserve for |
| 224 | // _all_ deopt and gc arguments, then start allocating. This |
| 225 | // will prevent some moves being inserted when vm state changes, |
| 226 | // but gc state doesn't between two calls. |
| 227 | return; |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 228 | } |
Igor Laevsky | 346ff62 | 2015-06-10 12:31:53 +0000 | [diff] [blame] | 229 | // Reserve this stack slot |
| 230 | Builder.StatepointLowering.reserveStackSlot(Offset); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 231 | |
Igor Laevsky | 346ff62 | 2015-06-10 12:31:53 +0000 | [diff] [blame] | 232 | // Cache this slot so we find it when going through the normal |
| 233 | // assignment loop. |
| 234 | SDValue Loc = Builder.DAG.getTargetFrameIndex(*Index, Incoming.getValueType()); |
| 235 | Builder.StatepointLowering.setLocation(Incoming, Loc); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 236 | } |
| 237 | |
| 238 | /// Remove any duplicate (as SDValues) from the derived pointer pairs. This |
| 239 | /// is not required for correctness. It's purpose is to reduce the size of |
| 240 | /// StackMap section. It has no effect on the number of spill slots required |
| 241 | /// or the actual lowering. |
| 242 | static void removeDuplicatesGCPtrs(SmallVectorImpl<const Value *> &Bases, |
| 243 | SmallVectorImpl<const Value *> &Ptrs, |
| 244 | SmallVectorImpl<const Value *> &Relocs, |
| 245 | SelectionDAGBuilder &Builder) { |
| 246 | |
Benjamin Kramer | df005cb | 2015-08-08 18:27:36 +0000 | [diff] [blame] | 247 | // This is horribly inefficient, but I don't care right now |
Matthias Braun | b30f2f51 | 2016-01-30 01:24:31 +0000 | [diff] [blame] | 248 | SmallSet<SDValue, 32> Seen; |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 249 | |
| 250 | SmallVector<const Value *, 64> NewBases, NewPtrs, NewRelocs; |
| 251 | for (size_t i = 0; i < Ptrs.size(); i++) { |
| 252 | SDValue SD = Builder.getValue(Ptrs[i]); |
| 253 | // Only add non-duplicates |
| 254 | if (Seen.count(SD) == 0) { |
| 255 | NewBases.push_back(Bases[i]); |
| 256 | NewPtrs.push_back(Ptrs[i]); |
| 257 | NewRelocs.push_back(Relocs[i]); |
| 258 | } |
| 259 | Seen.insert(SD); |
| 260 | } |
| 261 | assert(Bases.size() >= NewBases.size()); |
| 262 | assert(Ptrs.size() >= NewPtrs.size()); |
| 263 | assert(Relocs.size() >= NewRelocs.size()); |
| 264 | Bases = NewBases; |
| 265 | Ptrs = NewPtrs; |
| 266 | Relocs = NewRelocs; |
| 267 | assert(Ptrs.size() == Bases.size()); |
| 268 | assert(Ptrs.size() == Relocs.size()); |
| 269 | } |
| 270 | |
| 271 | /// Extract call from statepoint, lower it and return pointer to the |
| 272 | /// call node. Also update NodeMap so that getValue(statepoint) will |
| 273 | /// reference lowered call result |
Sanjoy Das | c6bf3e9 | 2015-05-06 02:36:20 +0000 | [diff] [blame] | 274 | static SDNode * |
Reid Kleckner | 51189f0a | 2015-09-08 23:28:38 +0000 | [diff] [blame] | 275 | lowerCallFromStatepoint(ImmutableStatepoint ISP, const BasicBlock *EHPadBB, |
Sanjoy Das | c6bf3e9 | 2015-05-06 02:36:20 +0000 | [diff] [blame] | 276 | SelectionDAGBuilder &Builder, |
| 277 | SmallVectorImpl<SDValue> &PendingExports) { |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 278 | |
Sanjoy Das | c6bf3e9 | 2015-05-06 02:36:20 +0000 | [diff] [blame] | 279 | ImmutableCallSite CS(ISP.getCallSite()); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 280 | |
Sanjoy Das | cfe41f0 | 2015-07-28 23:50:30 +0000 | [diff] [blame] | 281 | SDValue ActualCallee; |
| 282 | |
| 283 | if (ISP.getNumPatchBytes() > 0) { |
| 284 | // If we've been asked to emit a nop sequence instead of a call instruction |
| 285 | // for this statepoint then don't lower the call target, but use a constant |
| 286 | // `null` instead. Not lowering the call target lets statepoint clients get |
| 287 | // away without providing a physical address for the symbolic call target at |
| 288 | // link time. |
| 289 | |
| 290 | const auto &TLI = Builder.DAG.getTargetLoweringInfo(); |
| 291 | const auto &DL = Builder.DAG.getDataLayout(); |
| 292 | |
| 293 | unsigned AS = ISP.getCalledValue()->getType()->getPointerAddressSpace(); |
| 294 | ActualCallee = Builder.DAG.getConstant(0, Builder.getCurSDLoc(), |
| 295 | TLI.getPointerTy(DL, AS)); |
| 296 | } else |
| 297 | ActualCallee = Builder.getValue(ISP.getCalledValue()); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 298 | |
Sanjoy Das | c6bf3e9 | 2015-05-06 02:36:20 +0000 | [diff] [blame] | 299 | assert(CS.getCallingConv() != CallingConv::AnyReg && |
| 300 | "anyregcc is not supported on statepoints!"); |
| 301 | |
Sanjoy Das | 499d703 | 2015-05-06 02:36:26 +0000 | [diff] [blame] | 302 | Type *DefTy = ISP.getActualReturnType(); |
Sanjoy Das | c6bf3e9 | 2015-05-06 02:36:20 +0000 | [diff] [blame] | 303 | bool HasDef = !DefTy->isVoidTy(); |
| 304 | |
| 305 | SDValue ReturnValue, CallEndVal; |
| 306 | std::tie(ReturnValue, CallEndVal) = Builder.lowerCallOperands( |
Sanjoy Das | 4bfb472 | 2015-05-06 02:36:31 +0000 | [diff] [blame] | 307 | ISP.getCallSite(), ImmutableStatepoint::CallArgsBeginPos, |
Reid Kleckner | 51189f0a | 2015-09-08 23:28:38 +0000 | [diff] [blame] | 308 | ISP.getNumCallArgs(), ActualCallee, DefTy, EHPadBB, |
Sanjoy Das | 4bfb472 | 2015-05-06 02:36:31 +0000 | [diff] [blame] | 309 | false /* IsPatchPoint */); |
Sanjoy Das | c6bf3e9 | 2015-05-06 02:36:20 +0000 | [diff] [blame] | 310 | |
| 311 | SDNode *CallEnd = CallEndVal.getNode(); |
| 312 | |
| 313 | // Get a call instruction from the call sequence chain. Tail calls are not |
| 314 | // allowed. The following code is essentially reverse engineering X86's |
| 315 | // LowerCallTo. |
| 316 | // |
| 317 | // We are expecting DAG to have the following form: |
| 318 | // |
| 319 | // ch = eh_label (only in case of invoke statepoint) |
| 320 | // ch, glue = callseq_start ch |
| 321 | // ch, glue = X86::Call ch, glue |
| 322 | // ch, glue = callseq_end ch, glue |
| 323 | // get_return_value ch, glue |
| 324 | // |
Pat Gavlin | c8ea157 | 2015-11-17 16:04:21 +0000 | [diff] [blame] | 325 | // get_return_value can either be a sequence of CopyFromReg instructions |
| 326 | // to grab the return value from the return register(s), or it can be a LOAD |
| 327 | // to load a value returned by reference via a stack slot. |
Sanjoy Das | c6bf3e9 | 2015-05-06 02:36:20 +0000 | [diff] [blame] | 328 | |
Pat Gavlin | c8ea157 | 2015-11-17 16:04:21 +0000 | [diff] [blame] | 329 | if (HasDef) { |
| 330 | if (CallEnd->getOpcode() == ISD::LOAD) |
| 331 | CallEnd = CallEnd->getOperand(0).getNode(); |
| 332 | else |
| 333 | while (CallEnd->getOpcode() == ISD::CopyFromReg) |
| 334 | CallEnd = CallEnd->getOperand(0).getNode(); |
| 335 | } |
Sanjoy Das | c6bf3e9 | 2015-05-06 02:36:20 +0000 | [diff] [blame] | 336 | |
| 337 | assert(CallEnd->getOpcode() == ISD::CALLSEQ_END && "expected!"); |
| 338 | |
Igor Laevsky | 35fe692 | 2015-11-04 01:16:10 +0000 | [diff] [blame] | 339 | // Export the result value if needed |
| 340 | const Instruction *GCResult = ISP.getGCResult(); |
| 341 | if (HasDef && GCResult) { |
| 342 | if (GCResult->getParent() != CS.getParent()) { |
| 343 | // Result value will be used in a different basic block so we need to |
| 344 | // export it now. |
| 345 | // Default exporting mechanism will not work here because statepoint call |
| 346 | // has a different type than the actual call. It means that by default |
| 347 | // llvm will create export register of the wrong type (always i32 in our |
| 348 | // case). So instead we need to create export register with correct type |
| 349 | // manually. |
Igor Laevsky | 85f7f72 | 2015-03-10 16:26:48 +0000 | [diff] [blame] | 350 | // TODO: To eliminate this problem we can remove gc.result intrinsics |
Benjamin Kramer | df005cb | 2015-08-08 18:27:36 +0000 | [diff] [blame] | 351 | // completely and make statepoint call to return a tuple. |
Sanjoy Das | 499d703 | 2015-05-06 02:36:26 +0000 | [diff] [blame] | 352 | unsigned Reg = Builder.FuncInfo.CreateRegs(ISP.getActualReturnType()); |
Mehdi Amini | 56228da | 2015-07-09 01:57:34 +0000 | [diff] [blame] | 353 | RegsForValue RFV( |
| 354 | *Builder.DAG.getContext(), Builder.DAG.getTargetLoweringInfo(), |
| 355 | Builder.DAG.getDataLayout(), Reg, ISP.getActualReturnType()); |
Sanjoy Das | c6bf3e9 | 2015-05-06 02:36:20 +0000 | [diff] [blame] | 356 | SDValue Chain = Builder.DAG.getEntryNode(); |
| 357 | |
| 358 | RFV.getCopyToRegs(ReturnValue, Builder.DAG, Builder.getCurSDLoc(), Chain, |
| 359 | nullptr); |
| 360 | PendingExports.push_back(Chain); |
| 361 | Builder.FuncInfo.ValueMap[CS.getInstruction()] = Reg; |
Pat Gavlin | 022c5ac | 2015-04-29 21:52:45 +0000 | [diff] [blame] | 362 | } else { |
Igor Laevsky | 35fe692 | 2015-11-04 01:16:10 +0000 | [diff] [blame] | 363 | // Result value will be used in a same basic block. Don't export it or |
| 364 | // perform any explicit register copies. |
| 365 | // We'll replace the actuall call node shortly. gc_result will grab |
Igor Laevsky | 85f7f72 | 2015-03-10 16:26:48 +0000 | [diff] [blame] | 366 | // this value. |
Sanjoy Das | c6bf3e9 | 2015-05-06 02:36:20 +0000 | [diff] [blame] | 367 | Builder.setValue(CS.getInstruction(), ReturnValue); |
Igor Laevsky | 85f7f72 | 2015-03-10 16:26:48 +0000 | [diff] [blame] | 368 | } |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 369 | } else { |
| 370 | // The token value is never used from here on, just generate a poison value |
Sergey Dmitrouk | 842a51b | 2015-04-28 14:05:47 +0000 | [diff] [blame] | 371 | Builder.setValue(CS.getInstruction(), |
| 372 | Builder.DAG.getIntPtrConstant(-1, Builder.getCurSDLoc())); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 373 | } |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 374 | |
Sanjoy Das | c6bf3e9 | 2015-05-06 02:36:20 +0000 | [diff] [blame] | 375 | return CallEnd->getOperand(0).getNode(); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 376 | } |
| 377 | |
| 378 | /// Callect all gc pointers coming into statepoint intrinsic, clean them up, |
| 379 | /// and return two arrays: |
| 380 | /// Bases - base pointers incoming to this statepoint |
| 381 | /// Ptrs - derived pointers incoming to this statepoint |
| 382 | /// Relocs - the gc_relocate corresponding to each base/ptr pair |
| 383 | /// Elements of this arrays should be in one-to-one correspondence with each |
| 384 | /// other i.e Bases[i], Ptrs[i] are from the same gcrelocate call |
Pat Gavlin | 022c5ac | 2015-04-29 21:52:45 +0000 | [diff] [blame] | 385 | static void getIncomingStatepointGCValues( |
| 386 | SmallVectorImpl<const Value *> &Bases, SmallVectorImpl<const Value *> &Ptrs, |
| 387 | SmallVectorImpl<const Value *> &Relocs, ImmutableStatepoint StatepointSite, |
| 388 | SelectionDAGBuilder &Builder) { |
Manuel Jacob | 83eefa6 | 2016-01-05 04:03:00 +0000 | [diff] [blame] | 389 | for (const GCRelocateInst *Relocate : StatepointSite.getRelocates()) { |
| 390 | Relocs.push_back(Relocate); |
| 391 | Bases.push_back(Relocate->getBasePtr()); |
| 392 | Ptrs.push_back(Relocate->getDerivedPtr()); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 393 | } |
| 394 | |
| 395 | // Remove any redundant llvm::Values which map to the same SDValue as another |
| 396 | // input. Also has the effect of removing duplicates in the original |
| 397 | // llvm::Value input list as well. This is a useful optimization for |
| 398 | // reducing the size of the StackMap section. It has no other impact. |
| 399 | removeDuplicatesGCPtrs(Bases, Ptrs, Relocs, Builder); |
| 400 | |
| 401 | assert(Bases.size() == Ptrs.size() && Ptrs.size() == Relocs.size()); |
| 402 | } |
| 403 | |
| 404 | /// Spill a value incoming to the statepoint. It might be either part of |
| 405 | /// vmstate |
| 406 | /// or gcstate. In both cases unconditionally spill it on the stack unless it |
| 407 | /// is a null constant. Return pair with first element being frame index |
| 408 | /// containing saved value and second element with outgoing chain from the |
| 409 | /// emitted store |
| 410 | static std::pair<SDValue, SDValue> |
| 411 | spillIncomingStatepointValue(SDValue Incoming, SDValue Chain, |
| 412 | SelectionDAGBuilder &Builder) { |
| 413 | SDValue Loc = Builder.StatepointLowering.getLocation(Incoming); |
| 414 | |
| 415 | // Emit new store if we didn't do it for this ptr before |
| 416 | if (!Loc.getNode()) { |
| 417 | Loc = Builder.StatepointLowering.allocateStackSlot(Incoming.getValueType(), |
| 418 | Builder); |
| 419 | assert(isa<FrameIndexSDNode>(Loc)); |
| 420 | int Index = cast<FrameIndexSDNode>(Loc)->getIndex(); |
| 421 | // We use TargetFrameIndex so that isel will not select it into LEA |
| 422 | Loc = Builder.DAG.getTargetFrameIndex(Index, Incoming.getValueType()); |
| 423 | |
| 424 | // TODO: We can create TokenFactor node instead of |
| 425 | // chaining stores one after another, this may allow |
| 426 | // a bit more optimal scheduling for them |
| 427 | Chain = Builder.DAG.getStore(Chain, Builder.getCurSDLoc(), Incoming, Loc, |
Alex Lorenz | e40c8a2 | 2015-08-11 23:09:45 +0000 | [diff] [blame] | 428 | MachinePointerInfo::getFixedStack( |
| 429 | Builder.DAG.getMachineFunction(), Index), |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 430 | false, false, 0); |
| 431 | |
| 432 | Builder.StatepointLowering.setLocation(Incoming, Loc); |
| 433 | } |
| 434 | |
| 435 | assert(Loc.getNode()); |
| 436 | return std::make_pair(Loc, Chain); |
| 437 | } |
| 438 | |
| 439 | /// Lower a single value incoming to a statepoint node. This value can be |
| 440 | /// either a deopt value or a gc value, the handling is the same. We special |
| 441 | /// case constants and allocas, then fall back to spilling if required. |
| 442 | static void lowerIncomingStatepointValue(SDValue Incoming, |
| 443 | SmallVectorImpl<SDValue> &Ops, |
| 444 | SelectionDAGBuilder &Builder) { |
| 445 | SDValue Chain = Builder.getRoot(); |
| 446 | |
| 447 | if (ConstantSDNode *C = dyn_cast<ConstantSDNode>(Incoming)) { |
| 448 | // If the original value was a constant, make sure it gets recorded as |
| 449 | // such in the stackmap. This is required so that the consumer can |
| 450 | // parse any internal format to the deopt state. It also handles null |
Philip Reames | afdbcc6 | 2016-01-07 04:15:31 +0000 | [diff] [blame] | 451 | // pointers and other constant pointers in GC states. Note the constant |
| 452 | // vectors do not appear to actually hit this path and that anything larger |
| 453 | // than an i64 value (not type!) will fail asserts here. |
Pat Gavlin | c7dc6d6ee | 2015-05-12 19:50:19 +0000 | [diff] [blame] | 454 | pushStackMapConstant(Ops, Builder, C->getSExtValue()); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 455 | } else if (FrameIndexSDNode *FI = dyn_cast<FrameIndexSDNode>(Incoming)) { |
Philip Reames | f8f0933 | 2015-03-27 04:52:48 +0000 | [diff] [blame] | 456 | // This handles allocas as arguments to the statepoint (this is only |
| 457 | // really meaningful for a deopt value. For GC, we'd be trying to |
| 458 | // relocate the address of the alloca itself?) |
Pat Gavlin | 022c5ac | 2015-04-29 21:52:45 +0000 | [diff] [blame] | 459 | Ops.push_back(Builder.DAG.getTargetFrameIndex(FI->getIndex(), |
Philip Reames | f8f0933 | 2015-03-27 04:52:48 +0000 | [diff] [blame] | 460 | Incoming.getValueType())); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 461 | } else { |
| 462 | // Otherwise, locate a spill slot and explicitly spill it so it |
| 463 | // can be found by the runtime later. We currently do not support |
| 464 | // tracking values through callee saved registers to their eventual |
| 465 | // spill location. This would be a useful optimization, but would |
| 466 | // need to be optional since it requires a lot of complexity on the |
| 467 | // runtime side which not all would support. |
| 468 | std::pair<SDValue, SDValue> Res = |
| 469 | spillIncomingStatepointValue(Incoming, Chain, Builder); |
| 470 | Ops.push_back(Res.first); |
| 471 | Chain = Res.second; |
| 472 | } |
| 473 | |
| 474 | Builder.DAG.setRoot(Chain); |
| 475 | } |
| 476 | |
| 477 | /// Lower deopt state and gc pointer arguments of the statepoint. The actual |
| 478 | /// lowering is described in lowerIncomingStatepointValue. This function is |
| 479 | /// responsible for lowering everything in the right position and playing some |
| 480 | /// tricks to avoid redundant stack manipulation where possible. On |
| 481 | /// completion, 'Ops' will contain ready to use operands for machine code |
| 482 | /// statepoint. The chain nodes will have already been created and the DAG root |
| 483 | /// will be set to the last value spilled (if any were). |
| 484 | static void lowerStatepointMetaArgs(SmallVectorImpl<SDValue> &Ops, |
Igor Laevsky | 7fc58a4 | 2015-02-20 15:28:35 +0000 | [diff] [blame] | 485 | ImmutableStatepoint StatepointSite, |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 486 | SelectionDAGBuilder &Builder) { |
| 487 | |
| 488 | // Lower the deopt and gc arguments for this statepoint. Layout will |
| 489 | // be: deopt argument length, deopt arguments.., gc arguments... |
| 490 | |
| 491 | SmallVector<const Value *, 64> Bases, Ptrs, Relocations; |
Pat Gavlin | 022c5ac | 2015-04-29 21:52:45 +0000 | [diff] [blame] | 492 | getIncomingStatepointGCValues(Bases, Ptrs, Relocations, StatepointSite, |
| 493 | Builder); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 494 | |
Philip Reames | 4ac17a3 | 2015-01-07 19:07:50 +0000 | [diff] [blame] | 495 | #ifndef NDEBUG |
| 496 | // Check that each of the gc pointer and bases we've gotten out of the |
Philip Reames | 3e2cf53 | 2016-01-07 03:32:11 +0000 | [diff] [blame] | 497 | // safepoint is something the strategy thinks might be a pointer (or vector |
| 498 | // of pointers) into the GC heap. This is basically just here to help catch |
| 499 | // errors during statepoint insertion. TODO: This should actually be in the |
| 500 | // Verifier, but we can't get to the GCStrategy from there (yet). |
Philip Reames | e1bf270 | 2015-03-27 05:09:33 +0000 | [diff] [blame] | 501 | GCStrategy &S = Builder.GFI->getStrategy(); |
| 502 | for (const Value *V : Bases) { |
Philip Reames | 3e2cf53 | 2016-01-07 03:32:11 +0000 | [diff] [blame] | 503 | auto Opt = S.isGCManagedPointer(V->getType()->getScalarType()); |
Philip Reames | e1bf270 | 2015-03-27 05:09:33 +0000 | [diff] [blame] | 504 | if (Opt.hasValue()) { |
| 505 | assert(Opt.getValue() && |
| 506 | "non gc managed base pointer found in statepoint"); |
Philip Reames | 4ac17a3 | 2015-01-07 19:07:50 +0000 | [diff] [blame] | 507 | } |
Philip Reames | e1bf270 | 2015-03-27 05:09:33 +0000 | [diff] [blame] | 508 | } |
| 509 | for (const Value *V : Ptrs) { |
Philip Reames | 3e2cf53 | 2016-01-07 03:32:11 +0000 | [diff] [blame] | 510 | auto Opt = S.isGCManagedPointer(V->getType()->getScalarType()); |
Philip Reames | e1bf270 | 2015-03-27 05:09:33 +0000 | [diff] [blame] | 511 | if (Opt.hasValue()) { |
| 512 | assert(Opt.getValue() && |
| 513 | "non gc managed derived pointer found in statepoint"); |
Philip Reames | 4ac17a3 | 2015-01-07 19:07:50 +0000 | [diff] [blame] | 514 | } |
Philip Reames | e1bf270 | 2015-03-27 05:09:33 +0000 | [diff] [blame] | 515 | } |
| 516 | for (const Value *V : Relocations) { |
Philip Reames | 3e2cf53 | 2016-01-07 03:32:11 +0000 | [diff] [blame] | 517 | auto Opt = S.isGCManagedPointer(V->getType()->getScalarType()); |
Philip Reames | e1bf270 | 2015-03-27 05:09:33 +0000 | [diff] [blame] | 518 | if (Opt.hasValue()) { |
| 519 | assert(Opt.getValue() && "non gc managed pointer relocated"); |
Philip Reames | 4ac17a3 | 2015-01-07 19:07:50 +0000 | [diff] [blame] | 520 | } |
| 521 | } |
| 522 | #endif |
| 523 | |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 524 | // Before we actually start lowering (and allocating spill slots for values), |
| 525 | // reserve any stack slots which we judge to be profitable to reuse for a |
| 526 | // particular value. This is purely an optimization over the code below and |
| 527 | // doesn't change semantics at all. It is important for performance that we |
| 528 | // reserve slots for both deopt and gc values before lowering either. |
Pat Gavlin | 08d7027 | 2015-05-12 21:33:48 +0000 | [diff] [blame] | 529 | for (const Value *V : StatepointSite.vm_state_args()) { |
Igor Laevsky | 346ff62 | 2015-06-10 12:31:53 +0000 | [diff] [blame] | 530 | reservePreviousStackSlotForValue(V, Builder); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 531 | } |
Igor Laevsky | 87ef5ea | 2015-05-12 13:12:14 +0000 | [diff] [blame] | 532 | for (unsigned i = 0; i < Bases.size(); ++i) { |
Igor Laevsky | 346ff62 | 2015-06-10 12:31:53 +0000 | [diff] [blame] | 533 | reservePreviousStackSlotForValue(Bases[i], Builder); |
| 534 | reservePreviousStackSlotForValue(Ptrs[i], Builder); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 535 | } |
| 536 | |
| 537 | // First, prefix the list with the number of unique values to be |
| 538 | // lowered. Note that this is the number of *Values* not the |
| 539 | // number of SDValues required to lower them. |
Sanjoy Das | 499d703 | 2015-05-06 02:36:26 +0000 | [diff] [blame] | 540 | const int NumVMSArgs = StatepointSite.getNumTotalVMSArgs(); |
Pat Gavlin | c7dc6d6ee | 2015-05-12 19:50:19 +0000 | [diff] [blame] | 541 | pushStackMapConstant(Ops, Builder, NumVMSArgs); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 542 | |
Pat Gavlin | 08d7027 | 2015-05-12 21:33:48 +0000 | [diff] [blame] | 543 | assert(NumVMSArgs == std::distance(StatepointSite.vm_state_begin(), |
| 544 | StatepointSite.vm_state_end())); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 545 | |
| 546 | // The vm state arguments are lowered in an opaque manner. We do |
| 547 | // not know what type of values are contained within. We skip the |
| 548 | // first one since that happens to be the total number we lowered |
| 549 | // explicitly just above. We could have left it in the loop and |
| 550 | // not done it explicitly, but it's far easier to understand this |
| 551 | // way. |
Pat Gavlin | 08d7027 | 2015-05-12 21:33:48 +0000 | [diff] [blame] | 552 | for (const Value *V : StatepointSite.vm_state_args()) { |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 553 | SDValue Incoming = Builder.getValue(V); |
| 554 | lowerIncomingStatepointValue(Incoming, Ops, Builder); |
| 555 | } |
| 556 | |
| 557 | // Finally, go ahead and lower all the gc arguments. There's no prefixed |
| 558 | // length for this one. After lowering, we'll have the base and pointer |
| 559 | // arrays interwoven with each (lowered) base pointer immediately followed by |
| 560 | // it's (lowered) derived pointer. i.e |
| 561 | // (base[0], ptr[0], base[1], ptr[1], ...) |
Igor Laevsky | 87ef5ea | 2015-05-12 13:12:14 +0000 | [diff] [blame] | 562 | for (unsigned i = 0; i < Bases.size(); ++i) { |
| 563 | const Value *Base = Bases[i]; |
| 564 | lowerIncomingStatepointValue(Builder.getValue(Base), Ops, Builder); |
| 565 | |
| 566 | const Value *Ptr = Ptrs[i]; |
| 567 | lowerIncomingStatepointValue(Builder.getValue(Ptr), Ops, Builder); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 568 | } |
Philip Reames | f8f0933 | 2015-03-27 04:52:48 +0000 | [diff] [blame] | 569 | |
Pat Gavlin | 022c5ac | 2015-04-29 21:52:45 +0000 | [diff] [blame] | 570 | // If there are any explicit spill slots passed to the statepoint, record |
Philip Reames | f8f0933 | 2015-03-27 04:52:48 +0000 | [diff] [blame] | 571 | // them, but otherwise do not do anything special. These are user provided |
Pat Gavlin | 022c5ac | 2015-04-29 21:52:45 +0000 | [diff] [blame] | 572 | // allocas and give control over placement to the consumer. In this case, |
Philip Reames | f8f0933 | 2015-03-27 04:52:48 +0000 | [diff] [blame] | 573 | // it is the contents of the slot which may get updated, not the pointer to |
| 574 | // the alloca |
| 575 | for (Value *V : StatepointSite.gc_args()) { |
| 576 | SDValue Incoming = Builder.getValue(V); |
| 577 | if (FrameIndexSDNode *FI = dyn_cast<FrameIndexSDNode>(Incoming)) { |
| 578 | // This handles allocas as arguments to the statepoint |
Pat Gavlin | 022c5ac | 2015-04-29 21:52:45 +0000 | [diff] [blame] | 579 | Ops.push_back(Builder.DAG.getTargetFrameIndex(FI->getIndex(), |
Philip Reames | f8f0933 | 2015-03-27 04:52:48 +0000 | [diff] [blame] | 580 | Incoming.getValueType())); |
Philip Reames | f8f0933 | 2015-03-27 04:52:48 +0000 | [diff] [blame] | 581 | } |
| 582 | } |
Igor Laevsky | 423bc9ec | 2015-05-20 11:37:25 +0000 | [diff] [blame] | 583 | |
| 584 | // Record computed locations for all lowered values. |
| 585 | // This can not be embedded in lowering loops as we need to record *all* |
| 586 | // values, while previous loops account only values with unique SDValues. |
| 587 | const Instruction *StatepointInstr = |
| 588 | StatepointSite.getCallSite().getInstruction(); |
| 589 | FunctionLoweringInfo::StatepointSpilledValueMapTy &SpillMap = |
| 590 | Builder.FuncInfo.StatepointRelocatedValues[StatepointInstr]; |
| 591 | |
Manuel Jacob | 83eefa6 | 2016-01-05 04:03:00 +0000 | [diff] [blame] | 592 | for (const GCRelocateInst *Relocate : StatepointSite.getRelocates()) { |
| 593 | const Value *V = Relocate->getDerivedPtr(); |
Igor Laevsky | 423bc9ec | 2015-05-20 11:37:25 +0000 | [diff] [blame] | 594 | SDValue SDV = Builder.getValue(V); |
| 595 | SDValue Loc = Builder.StatepointLowering.getLocation(SDV); |
| 596 | |
| 597 | if (Loc.getNode()) { |
| 598 | SpillMap[V] = cast<FrameIndexSDNode>(Loc)->getIndex(); |
| 599 | } else { |
| 600 | // Record value as visited, but not spilled. This is case for allocas |
Benjamin Kramer | df005cb | 2015-08-08 18:27:36 +0000 | [diff] [blame] | 601 | // and constants. For this values we can avoid emitting spill load while |
Igor Laevsky | 423bc9ec | 2015-05-20 11:37:25 +0000 | [diff] [blame] | 602 | // visiting corresponding gc_relocate. |
| 603 | // Actually we do not need to record them in this map at all. |
Benjamin Kramer | df005cb | 2015-08-08 18:27:36 +0000 | [diff] [blame] | 604 | // We do this only to check that we are not relocating any unvisited |
| 605 | // value. |
Igor Laevsky | 423bc9ec | 2015-05-20 11:37:25 +0000 | [diff] [blame] | 606 | SpillMap[V] = None; |
| 607 | |
| 608 | // Default llvm mechanisms for exporting values which are used in |
| 609 | // different basic blocks does not work for gc relocates. |
| 610 | // Note that it would be incorrect to teach llvm that all relocates are |
Benjamin Kramer | df005cb | 2015-08-08 18:27:36 +0000 | [diff] [blame] | 611 | // uses of the corresponding values so that it would automatically |
Igor Laevsky | 423bc9ec | 2015-05-20 11:37:25 +0000 | [diff] [blame] | 612 | // export them. Relocates of the spilled values does not use original |
| 613 | // value. |
Manuel Jacob | 83eefa6 | 2016-01-05 04:03:00 +0000 | [diff] [blame] | 614 | if (Relocate->getParent() != StatepointInstr->getParent()) |
Igor Laevsky | 423bc9ec | 2015-05-20 11:37:25 +0000 | [diff] [blame] | 615 | Builder.ExportFromCurrentBlock(V); |
| 616 | } |
| 617 | } |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 618 | } |
Igor Laevsky | 7fc58a4 | 2015-02-20 15:28:35 +0000 | [diff] [blame] | 619 | |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 620 | void SelectionDAGBuilder::visitStatepoint(const CallInst &CI) { |
Igor Laevsky | 7fc58a4 | 2015-02-20 15:28:35 +0000 | [diff] [blame] | 621 | // Check some preconditions for sanity |
| 622 | assert(isStatepoint(&CI) && |
| 623 | "function called must be the statepoint function"); |
| 624 | |
| 625 | LowerStatepoint(ImmutableStatepoint(&CI)); |
| 626 | } |
| 627 | |
Pat Gavlin | 022c5ac | 2015-04-29 21:52:45 +0000 | [diff] [blame] | 628 | void SelectionDAGBuilder::LowerStatepoint( |
Reid Kleckner | 51189f0a | 2015-09-08 23:28:38 +0000 | [diff] [blame] | 629 | ImmutableStatepoint ISP, const BasicBlock *EHPadBB /*= nullptr*/) { |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 630 | // The basic scheme here is that information about both the original call and |
| 631 | // the safepoint is encoded in the CallInst. We create a temporary call and |
| 632 | // lower it, then reverse engineer the calling sequence. |
| 633 | |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 634 | NumOfStatepoints++; |
| 635 | // Clear state |
| 636 | StatepointLowering.startNewStatepoint(*this); |
| 637 | |
Igor Laevsky | 7fc58a4 | 2015-02-20 15:28:35 +0000 | [diff] [blame] | 638 | ImmutableCallSite CS(ISP.getCallSite()); |
| 639 | |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 640 | #ifndef NDEBUG |
Igor Laevsky | 35fe692 | 2015-11-04 01:16:10 +0000 | [diff] [blame] | 641 | // Consistency check. Check only relocates in the same basic block as thier |
| 642 | // statepoint. |
| 643 | for (const User *U : CS->users()) { |
| 644 | const CallInst *Call = cast<CallInst>(U); |
Manuel Jacob | 83eefa6 | 2016-01-05 04:03:00 +0000 | [diff] [blame] | 645 | if (isa<GCRelocateInst>(Call) && Call->getParent() == CS.getParent()) |
Igor Laevsky | 35fe692 | 2015-11-04 01:16:10 +0000 | [diff] [blame] | 646 | StatepointLowering.scheduleRelocCall(*Call); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 647 | } |
| 648 | #endif |
| 649 | |
Philip Reames | 72fbe7a | 2014-12-02 21:01:48 +0000 | [diff] [blame] | 650 | #ifndef NDEBUG |
| 651 | // If this is a malformed statepoint, report it early to simplify debugging. |
| 652 | // This should catch any IR level mistake that's made when constructing or |
| 653 | // transforming statepoints. |
| 654 | ISP.verify(); |
Philip Reames | 4ac17a3 | 2015-01-07 19:07:50 +0000 | [diff] [blame] | 655 | |
| 656 | // Check that the associated GCStrategy expects to encounter statepoints. |
Philip Reames | e1bf270 | 2015-03-27 05:09:33 +0000 | [diff] [blame] | 657 | assert(GFI->getStrategy().useStatepoints() && |
| 658 | "GCStrategy does not expect to encounter statepoints"); |
Philip Reames | 72fbe7a | 2014-12-02 21:01:48 +0000 | [diff] [blame] | 659 | #endif |
| 660 | |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 661 | // Lower statepoint vmstate and gcstate arguments |
Sanjoy Das | 3fb91c0 | 2015-05-05 23:06:49 +0000 | [diff] [blame] | 662 | SmallVector<SDValue, 10> LoweredMetaArgs; |
| 663 | lowerStatepointMetaArgs(LoweredMetaArgs, ISP, *this); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 664 | |
| 665 | // Get call node, we will replace it later with statepoint |
Sanjoy Das | c6bf3e9 | 2015-05-06 02:36:20 +0000 | [diff] [blame] | 666 | SDNode *CallNode = |
Reid Kleckner | 51189f0a | 2015-09-08 23:28:38 +0000 | [diff] [blame] | 667 | lowerCallFromStatepoint(ISP, EHPadBB, *this, PendingExports); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 668 | |
Pat Gavlin | cc0431d | 2015-05-08 18:07:42 +0000 | [diff] [blame] | 669 | // Construct the actual GC_TRANSITION_START, STATEPOINT, and GC_TRANSITION_END |
| 670 | // nodes with all the appropriate arguments and return values. |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 671 | |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 672 | // Call Node: Chain, Target, {Args}, RegMask, [Glue] |
Pat Gavlin | cc0431d | 2015-05-08 18:07:42 +0000 | [diff] [blame] | 673 | SDValue Chain = CallNode->getOperand(0); |
| 674 | |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 675 | SDValue Glue; |
Pat Gavlin | cc0431d | 2015-05-08 18:07:42 +0000 | [diff] [blame] | 676 | bool CallHasIncomingGlue = CallNode->getGluedNode(); |
| 677 | if (CallHasIncomingGlue) { |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 678 | // Glue is always last operand |
| 679 | Glue = CallNode->getOperand(CallNode->getNumOperands() - 1); |
| 680 | } |
Pat Gavlin | cc0431d | 2015-05-08 18:07:42 +0000 | [diff] [blame] | 681 | |
| 682 | // Build the GC_TRANSITION_START node if necessary. |
| 683 | // |
| 684 | // The operands to the GC_TRANSITION_{START,END} nodes are laid out in the |
| 685 | // order in which they appear in the call to the statepoint intrinsic. If |
| 686 | // any of the operands is a pointer-typed, that operand is immediately |
| 687 | // followed by a SRCVALUE for the pointer that may be used during lowering |
| 688 | // (e.g. to form MachinePointerInfo values for loads/stores). |
| 689 | const bool IsGCTransition = |
| 690 | (ISP.getFlags() & (uint64_t)StatepointFlags::GCTransition) == |
| 691 | (uint64_t)StatepointFlags::GCTransition; |
| 692 | if (IsGCTransition) { |
| 693 | SmallVector<SDValue, 8> TSOps; |
| 694 | |
| 695 | // Add chain |
| 696 | TSOps.push_back(Chain); |
| 697 | |
| 698 | // Add GC transition arguments |
Pat Gavlin | 08d7027 | 2015-05-12 21:33:48 +0000 | [diff] [blame] | 699 | for (const Value *V : ISP.gc_transition_args()) { |
| 700 | TSOps.push_back(getValue(V)); |
| 701 | if (V->getType()->isPointerTy()) |
| 702 | TSOps.push_back(DAG.getSrcValue(V)); |
Pat Gavlin | cc0431d | 2015-05-08 18:07:42 +0000 | [diff] [blame] | 703 | } |
| 704 | |
| 705 | // Add glue if necessary |
| 706 | if (CallHasIncomingGlue) |
| 707 | TSOps.push_back(Glue); |
| 708 | |
| 709 | SDVTList NodeTys = DAG.getVTList(MVT::Other, MVT::Glue); |
| 710 | |
| 711 | SDValue GCTransitionStart = |
| 712 | DAG.getNode(ISD::GC_TRANSITION_START, getCurSDLoc(), NodeTys, TSOps); |
| 713 | |
| 714 | Chain = GCTransitionStart.getValue(0); |
| 715 | Glue = GCTransitionStart.getValue(1); |
| 716 | } |
| 717 | |
Sanjoy Das | a1d39ba | 2015-05-12 23:52:24 +0000 | [diff] [blame] | 718 | // TODO: Currently, all of these operands are being marked as read/write in |
| 719 | // PrologEpilougeInserter.cpp, we should special case the VMState arguments |
| 720 | // and flags to be read-only. |
| 721 | SmallVector<SDValue, 40> Ops; |
| 722 | |
| 723 | // Add the <id> and <numBytes> constants. |
| 724 | Ops.push_back(DAG.getTargetConstant(ISP.getID(), getCurSDLoc(), MVT::i64)); |
| 725 | Ops.push_back( |
| 726 | DAG.getTargetConstant(ISP.getNumPatchBytes(), getCurSDLoc(), MVT::i32)); |
| 727 | |
Pat Gavlin | cc0431d | 2015-05-08 18:07:42 +0000 | [diff] [blame] | 728 | // Calculate and push starting position of vmstate arguments |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 729 | // Get number of arguments incoming directly into call node |
| 730 | unsigned NumCallRegArgs = |
Pat Gavlin | cc0431d | 2015-05-08 18:07:42 +0000 | [diff] [blame] | 731 | CallNode->getNumOperands() - (CallHasIncomingGlue ? 4 : 3); |
Sergey Dmitrouk | 842a51b | 2015-04-28 14:05:47 +0000 | [diff] [blame] | 732 | Ops.push_back(DAG.getTargetConstant(NumCallRegArgs, getCurSDLoc(), MVT::i32)); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 733 | |
| 734 | // Add call target |
| 735 | SDValue CallTarget = SDValue(CallNode->getOperand(1).getNode(), 0); |
| 736 | Ops.push_back(CallTarget); |
| 737 | |
| 738 | // Add call arguments |
| 739 | // Get position of register mask in the call |
| 740 | SDNode::op_iterator RegMaskIt; |
Pat Gavlin | cc0431d | 2015-05-08 18:07:42 +0000 | [diff] [blame] | 741 | if (CallHasIncomingGlue) |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 742 | RegMaskIt = CallNode->op_end() - 2; |
| 743 | else |
| 744 | RegMaskIt = CallNode->op_end() - 1; |
| 745 | Ops.insert(Ops.end(), CallNode->op_begin() + 2, RegMaskIt); |
| 746 | |
Pat Gavlin | c7dc6d6ee | 2015-05-12 19:50:19 +0000 | [diff] [blame] | 747 | // Add a constant argument for the calling convention |
| 748 | pushStackMapConstant(Ops, *this, CS.getCallingConv()); |
| 749 | |
| 750 | // Add a constant argument for the flags |
Sanjoy Das | a1d39ba | 2015-05-12 23:52:24 +0000 | [diff] [blame] | 751 | uint64_t Flags = ISP.getFlags(); |
Pat Gavlin | cc0431d | 2015-05-08 18:07:42 +0000 | [diff] [blame] | 752 | assert( |
| 753 | ((Flags & ~(uint64_t)StatepointFlags::MaskAll) == 0) |
| 754 | && "unknown flag used"); |
Pat Gavlin | c7dc6d6ee | 2015-05-12 19:50:19 +0000 | [diff] [blame] | 755 | pushStackMapConstant(Ops, *this, Flags); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 756 | |
| 757 | // Insert all vmstate and gcstate arguments |
Sanjoy Das | 3fb91c0 | 2015-05-05 23:06:49 +0000 | [diff] [blame] | 758 | Ops.insert(Ops.end(), LoweredMetaArgs.begin(), LoweredMetaArgs.end()); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 759 | |
| 760 | // Add register mask from call node |
| 761 | Ops.push_back(*RegMaskIt); |
| 762 | |
| 763 | // Add chain |
Pat Gavlin | cc0431d | 2015-05-08 18:07:42 +0000 | [diff] [blame] | 764 | Ops.push_back(Chain); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 765 | |
| 766 | // Same for the glue, but we add it only if original call had it |
| 767 | if (Glue.getNode()) |
| 768 | Ops.push_back(Glue); |
| 769 | |
Benjamin Kramer | ea68a94 | 2015-02-19 15:26:17 +0000 | [diff] [blame] | 770 | // Compute return values. Provide a glue output since we consume one as |
| 771 | // input. This allows someone else to chain off us as needed. |
| 772 | SDVTList NodeTys = DAG.getVTList(MVT::Other, MVT::Glue); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 773 | |
Pat Gavlin | 022c5ac | 2015-04-29 21:52:45 +0000 | [diff] [blame] | 774 | SDNode *StatepointMCNode = |
| 775 | DAG.getMachineNode(TargetOpcode::STATEPOINT, getCurSDLoc(), NodeTys, Ops); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 776 | |
Pat Gavlin | cc0431d | 2015-05-08 18:07:42 +0000 | [diff] [blame] | 777 | SDNode *SinkNode = StatepointMCNode; |
| 778 | |
| 779 | // Build the GC_TRANSITION_END node if necessary. |
| 780 | // |
| 781 | // See the comment above regarding GC_TRANSITION_START for the layout of |
| 782 | // the operands to the GC_TRANSITION_END node. |
| 783 | if (IsGCTransition) { |
| 784 | SmallVector<SDValue, 8> TEOps; |
| 785 | |
| 786 | // Add chain |
| 787 | TEOps.push_back(SDValue(StatepointMCNode, 0)); |
| 788 | |
| 789 | // Add GC transition arguments |
Pat Gavlin | 08d7027 | 2015-05-12 21:33:48 +0000 | [diff] [blame] | 790 | for (const Value *V : ISP.gc_transition_args()) { |
| 791 | TEOps.push_back(getValue(V)); |
| 792 | if (V->getType()->isPointerTy()) |
| 793 | TEOps.push_back(DAG.getSrcValue(V)); |
Pat Gavlin | cc0431d | 2015-05-08 18:07:42 +0000 | [diff] [blame] | 794 | } |
| 795 | |
| 796 | // Add glue |
| 797 | TEOps.push_back(SDValue(StatepointMCNode, 1)); |
| 798 | |
| 799 | SDVTList NodeTys = DAG.getVTList(MVT::Other, MVT::Glue); |
| 800 | |
| 801 | SDValue GCTransitionStart = |
| 802 | DAG.getNode(ISD::GC_TRANSITION_END, getCurSDLoc(), NodeTys, TEOps); |
| 803 | |
| 804 | SinkNode = GCTransitionStart.getNode(); |
| 805 | } |
| 806 | |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 807 | // Replace original call |
Pat Gavlin | cc0431d | 2015-05-08 18:07:42 +0000 | [diff] [blame] | 808 | DAG.ReplaceAllUsesWith(CallNode, SinkNode); // This may update Root |
Benjamin Kramer | df005cb | 2015-08-08 18:27:36 +0000 | [diff] [blame] | 809 | // Remove original call node |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 810 | DAG.DeleteNode(CallNode); |
| 811 | |
| 812 | // DON'T set the root - under the assumption that it's already set past the |
| 813 | // inserted node we created. |
| 814 | |
| 815 | // TODO: A better future implementation would be to emit a single variable |
| 816 | // argument, variable return value STATEPOINT node here and then hookup the |
| 817 | // return value of each gc.relocate to the respective output of the |
| 818 | // previously emitted STATEPOINT value. Unfortunately, this doesn't appear |
| 819 | // to actually be possible today. |
| 820 | } |
| 821 | |
| 822 | void SelectionDAGBuilder::visitGCResult(const CallInst &CI) { |
| 823 | // The result value of the gc_result is simply the result of the actual |
| 824 | // call. We've already emitted this, so just grab the value. |
| 825 | Instruction *I = cast<Instruction>(CI.getArgOperand(0)); |
Pat Gavlin | 022c5ac | 2015-04-29 21:52:45 +0000 | [diff] [blame] | 826 | assert(isStatepoint(I) && "first argument must be a statepoint token"); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 827 | |
Igor Laevsky | 35fe692 | 2015-11-04 01:16:10 +0000 | [diff] [blame] | 828 | if (I->getParent() != CI.getParent()) { |
| 829 | // Statepoint is in different basic block so we should have stored call |
| 830 | // result in a virtual register. |
Igor Laevsky | 85f7f72 | 2015-03-10 16:26:48 +0000 | [diff] [blame] | 831 | // We can not use default getValue() functionality to copy value from this |
| 832 | // register because statepoint and actuall call return types can be |
| 833 | // different, and getValue() will use CopyFromReg of the wrong type, |
| 834 | // which is always i32 in our case. |
Sanjoy Das | bbb2e82 | 2015-07-02 02:53:45 +0000 | [diff] [blame] | 835 | PointerType *CalleeType = cast<PointerType>( |
| 836 | ImmutableStatepoint(I).getCalledValue()->getType()); |
Pat Gavlin | 022c5ac | 2015-04-29 21:52:45 +0000 | [diff] [blame] | 837 | Type *RetTy = |
| 838 | cast<FunctionType>(CalleeType->getElementType())->getReturnType(); |
Igor Laevsky | 85f7f72 | 2015-03-10 16:26:48 +0000 | [diff] [blame] | 839 | SDValue CopyFromReg = getCopyFromRegs(I, RetTy); |
| 840 | |
| 841 | assert(CopyFromReg.getNode()); |
| 842 | setValue(&CI, CopyFromReg); |
Pat Gavlin | 022c5ac | 2015-04-29 21:52:45 +0000 | [diff] [blame] | 843 | } else { |
Igor Laevsky | 85f7f72 | 2015-03-10 16:26:48 +0000 | [diff] [blame] | 844 | setValue(&CI, getValue(I)); |
| 845 | } |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 846 | } |
| 847 | |
Manuel Jacob | 83eefa6 | 2016-01-05 04:03:00 +0000 | [diff] [blame] | 848 | void SelectionDAGBuilder::visitGCRelocate(const GCRelocateInst &Relocate) { |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 849 | #ifndef NDEBUG |
| 850 | // Consistency check |
Igor Laevsky | 35fe692 | 2015-11-04 01:16:10 +0000 | [diff] [blame] | 851 | // We skip this check for relocates not in the same basic block as thier |
| 852 | // statepoint. It would be too expensive to preserve validation info through |
| 853 | // different basic blocks. |
Manuel Jacob | 83eefa6 | 2016-01-05 04:03:00 +0000 | [diff] [blame] | 854 | if (Relocate.getStatepoint()->getParent() == Relocate.getParent()) { |
| 855 | StatepointLowering.relocCallVisited(Relocate); |
Igor Laevsky | 423bc9ec | 2015-05-20 11:37:25 +0000 | [diff] [blame] | 856 | } |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 857 | #endif |
| 858 | |
Manuel Jacob | 83eefa6 | 2016-01-05 04:03:00 +0000 | [diff] [blame] | 859 | const Value *DerivedPtr = Relocate.getDerivedPtr(); |
Igor Laevsky | 423bc9ec | 2015-05-20 11:37:25 +0000 | [diff] [blame] | 860 | SDValue SD = getValue(DerivedPtr); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 861 | |
Igor Laevsky | 423bc9ec | 2015-05-20 11:37:25 +0000 | [diff] [blame] | 862 | FunctionLoweringInfo::StatepointSpilledValueMapTy &SpillMap = |
Manuel Jacob | 83eefa6 | 2016-01-05 04:03:00 +0000 | [diff] [blame] | 863 | FuncInfo.StatepointRelocatedValues[Relocate.getStatepoint()]; |
Igor Laevsky | 423bc9ec | 2015-05-20 11:37:25 +0000 | [diff] [blame] | 864 | |
| 865 | // We should have recorded location for this pointer |
| 866 | assert(SpillMap.count(DerivedPtr) && "Relocating not lowered gc value"); |
| 867 | Optional<int> DerivedPtrLocation = SpillMap[DerivedPtr]; |
| 868 | |
| 869 | // We didn't need to spill these special cases (constants and allocas). |
| 870 | // See the handling in spillIncomingValueForStatepoint for detail. |
| 871 | if (!DerivedPtrLocation) { |
Manuel Jacob | 83eefa6 | 2016-01-05 04:03:00 +0000 | [diff] [blame] | 872 | setValue(&Relocate, SD); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 873 | return; |
| 874 | } |
| 875 | |
Igor Laevsky | 423bc9ec | 2015-05-20 11:37:25 +0000 | [diff] [blame] | 876 | SDValue SpillSlot = DAG.getTargetFrameIndex(*DerivedPtrLocation, |
| 877 | SD.getValueType()); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 878 | |
Igor Laevsky | 423bc9ec | 2015-05-20 11:37:25 +0000 | [diff] [blame] | 879 | // Be conservative: flush all pending loads |
| 880 | // TODO: Probably we can be less restrictive on this, |
Benjamin Kramer | df005cb | 2015-08-08 18:27:36 +0000 | [diff] [blame] | 881 | // it may allow more scheduling opportunities. |
Igor Laevsky | 423bc9ec | 2015-05-20 11:37:25 +0000 | [diff] [blame] | 882 | SDValue Chain = getRoot(); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 883 | |
Igor Laevsky | 423bc9ec | 2015-05-20 11:37:25 +0000 | [diff] [blame] | 884 | SDValue SpillLoad = |
Alex Lorenz | e40c8a2 | 2015-08-11 23:09:45 +0000 | [diff] [blame] | 885 | DAG.getLoad(SpillSlot.getValueType(), getCurSDLoc(), Chain, SpillSlot, |
| 886 | MachinePointerInfo::getFixedStack(DAG.getMachineFunction(), |
| 887 | *DerivedPtrLocation), |
| 888 | false, false, false, 0); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 889 | |
Igor Laevsky | 423bc9ec | 2015-05-20 11:37:25 +0000 | [diff] [blame] | 890 | // Again, be conservative, don't emit pending loads |
| 891 | DAG.setRoot(SpillLoad.getValue(1)); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 892 | |
Igor Laevsky | 423bc9ec | 2015-05-20 11:37:25 +0000 | [diff] [blame] | 893 | assert(SpillLoad.getNode()); |
Manuel Jacob | 83eefa6 | 2016-01-05 04:03:00 +0000 | [diff] [blame] | 894 | setValue(&Relocate, SpillLoad); |
Philip Reames | 1a1bdb2 | 2014-12-02 18:50:36 +0000 | [diff] [blame] | 895 | } |