| Eugene Zelenko | db56e5a | 2017-02-22 22:32:51 +0000 | [diff] [blame] | 1 | //===- StackProtector.cpp - Stack Protector Insertion ---------------------===// |
| Bill Wendling | 05d8417 | 2008-11-04 02:10:20 +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 | // |
| Bill Wendling | 64adc71 | 2008-11-04 21:53:09 +0000 | [diff] [blame] | 10 | // This pass inserts stack protectors into functions which need them. A variable |
| 11 | // with a random value in it is stored onto the stack before the local variables |
| 12 | // are allocated. Upon exiting the block, the stored value is checked. If it's |
| Bill Wendling | 05d8417 | 2008-11-04 02:10:20 +0000 | [diff] [blame] | 13 | // changed, then there was some sort of violation and the program aborts. |
| 14 | // |
| 15 | //===----------------------------------------------------------------------===// |
| 16 | |
| Bill Wendling | 7c8f96a | 2013-01-23 06:43:53 +0000 | [diff] [blame] | 17 | #include "llvm/ADT/SmallPtrSet.h" |
| 18 | #include "llvm/ADT/Statistic.h" |
| Akira Hatanaka | b9991a2 | 2014-12-01 04:27:03 +0000 | [diff] [blame] | 19 | #include "llvm/Analysis/BranchProbabilityInfo.h" |
| Etienne Bergeron | 22bfa83 | 2016-06-07 20:15:35 +0000 | [diff] [blame] | 20 | #include "llvm/Analysis/EHPersonalities.h" |
| Adam Nemet | 0965da2 | 2017-10-09 23:19:02 +0000 | [diff] [blame] | 21 | #include "llvm/Analysis/OptimizationRemarkEmitter.h" |
| Chandler Carruth | 8a8cd2b | 2014-01-07 11:48:04 +0000 | [diff] [blame] | 22 | #include "llvm/CodeGen/Passes.h" |
| Eugene Zelenko | 6ac7a34 | 2017-06-07 23:53:32 +0000 | [diff] [blame] | 23 | #include "llvm/CodeGen/StackProtector.h" |
| Matthias Braun | e89461a | 2017-06-05 22:59:21 +0000 | [diff] [blame] | 24 | #include "llvm/CodeGen/TargetPassConfig.h" |
| Chandler Carruth | 9fb823b | 2013-01-02 11:36:10 +0000 | [diff] [blame] | 25 | #include "llvm/IR/Attributes.h" |
| Eugene Zelenko | db56e5a | 2017-02-22 22:32:51 +0000 | [diff] [blame] | 26 | #include "llvm/IR/BasicBlock.h" |
| Chandler Carruth | 9fb823b | 2013-01-02 11:36:10 +0000 | [diff] [blame] | 27 | #include "llvm/IR/Constants.h" |
| 28 | #include "llvm/IR/DataLayout.h" |
| Yunzhong Gao | b386955 | 2016-06-30 18:49:04 +0000 | [diff] [blame] | 29 | #include "llvm/IR/DebugInfo.h" |
| Eugene Zelenko | db56e5a | 2017-02-22 22:32:51 +0000 | [diff] [blame] | 30 | #include "llvm/IR/DebugLoc.h" |
| Chandler Carruth | 9fb823b | 2013-01-02 11:36:10 +0000 | [diff] [blame] | 31 | #include "llvm/IR/DerivedTypes.h" |
| Eugene Zelenko | 6ac7a34 | 2017-06-07 23:53:32 +0000 | [diff] [blame] | 32 | #include "llvm/IR/Dominators.h" |
| Chandler Carruth | 9fb823b | 2013-01-02 11:36:10 +0000 | [diff] [blame] | 33 | #include "llvm/IR/Function.h" |
| Benjamin Kramer | d93817f | 2013-09-09 17:38:01 +0000 | [diff] [blame] | 34 | #include "llvm/IR/IRBuilder.h" |
| Eugene Zelenko | db56e5a | 2017-02-22 22:32:51 +0000 | [diff] [blame] | 35 | #include "llvm/IR/Instruction.h" |
| Chandler Carruth | 9fb823b | 2013-01-02 11:36:10 +0000 | [diff] [blame] | 36 | #include "llvm/IR/Instructions.h" |
| 37 | #include "llvm/IR/Intrinsics.h" |
| Akira Hatanaka | b9991a2 | 2014-12-01 04:27:03 +0000 | [diff] [blame] | 38 | #include "llvm/IR/MDBuilder.h" |
| Chandler Carruth | 9fb823b | 2013-01-02 11:36:10 +0000 | [diff] [blame] | 39 | #include "llvm/IR/Module.h" |
| Eugene Zelenko | db56e5a | 2017-02-22 22:32:51 +0000 | [diff] [blame] | 40 | #include "llvm/IR/Type.h" |
| 41 | #include "llvm/IR/User.h" |
| 42 | #include "llvm/Pass.h" |
| 43 | #include "llvm/Support/Casting.h" |
| Bill Wendling | 05d8417 | 2008-11-04 02:10:20 +0000 | [diff] [blame] | 44 | #include "llvm/Support/CommandLine.h" |
| Eugene Zelenko | db56e5a | 2017-02-22 22:32:51 +0000 | [diff] [blame] | 45 | #include "llvm/Target/TargetLowering.h" |
| 46 | #include "llvm/Target/TargetMachine.h" |
| 47 | #include "llvm/Target/TargetOptions.h" |
| Eric Christopher | d913448 | 2014-08-04 21:25:23 +0000 | [diff] [blame] | 48 | #include "llvm/Target/TargetSubtargetInfo.h" |
| Eugene Zelenko | db56e5a | 2017-02-22 22:32:51 +0000 | [diff] [blame] | 49 | #include <utility> |
| 50 | |
| Bill Wendling | 05d8417 | 2008-11-04 02:10:20 +0000 | [diff] [blame] | 51 | using namespace llvm; |
| 52 | |
| Chandler Carruth | 1b9dde0 | 2014-04-22 02:02:50 +0000 | [diff] [blame] | 53 | #define DEBUG_TYPE "stack-protector" |
| 54 | |
| Bill Wendling | 7c8f96a | 2013-01-23 06:43:53 +0000 | [diff] [blame] | 55 | STATISTIC(NumFunProtected, "Number of functions protected"); |
| 56 | STATISTIC(NumAddrTaken, "Number of local variables that have their address" |
| 57 | " taken."); |
| 58 | |
| Josh Magee | 7245f1d | 2013-10-30 02:25:14 +0000 | [diff] [blame] | 59 | static cl::opt<bool> EnableSelectionDAGSP("enable-selectiondag-sp", |
| 60 | cl::init(true), cl::Hidden); |
| Michael Gottesman | 5e57068 | 2013-08-20 08:36:53 +0000 | [diff] [blame] | 61 | |
| Bill Wendling | 05d8417 | 2008-11-04 02:10:20 +0000 | [diff] [blame] | 62 | char StackProtector::ID = 0; |
| Eugene Zelenko | 6ac7a34 | 2017-06-07 23:53:32 +0000 | [diff] [blame] | 63 | |
| Matthias Braun | 1527baa | 2017-05-25 21:26:32 +0000 | [diff] [blame] | 64 | INITIALIZE_PASS_BEGIN(StackProtector, DEBUG_TYPE, |
| Francis Visoiu Mistrih | 8b61764 | 2017-05-18 17:21:13 +0000 | [diff] [blame] | 65 | "Insert stack protectors", false, true) |
| 66 | INITIALIZE_PASS_DEPENDENCY(TargetPassConfig) |
| Matthias Braun | 1527baa | 2017-05-25 21:26:32 +0000 | [diff] [blame] | 67 | INITIALIZE_PASS_END(StackProtector, DEBUG_TYPE, |
| Francis Visoiu Mistrih | 8b61764 | 2017-05-18 17:21:13 +0000 | [diff] [blame] | 68 | "Insert stack protectors", false, true) |
| Bill Wendling | 05d8417 | 2008-11-04 02:10:20 +0000 | [diff] [blame] | 69 | |
| Francis Visoiu Mistrih | 8b61764 | 2017-05-18 17:21:13 +0000 | [diff] [blame] | 70 | FunctionPass *llvm::createStackProtectorPass() { return new StackProtector(); } |
| Bill Wendling | 05d8417 | 2008-11-04 02:10:20 +0000 | [diff] [blame] | 71 | |
| Josh Magee | 7245f1d | 2013-10-30 02:25:14 +0000 | [diff] [blame] | 72 | StackProtector::SSPLayoutKind |
| 73 | StackProtector::getSSPLayout(const AllocaInst *AI) const { |
| Josh Magee | 3f1c0e3 | 2013-10-29 21:16:16 +0000 | [diff] [blame] | 74 | return AI ? Layout.lookup(AI) : SSPLK_None; |
| 75 | } |
| 76 | |
| Hal Finkel | a69e5b8 | 2014-01-20 19:49:14 +0000 | [diff] [blame] | 77 | void StackProtector::adjustForColoring(const AllocaInst *From, |
| 78 | const AllocaInst *To) { |
| 79 | // When coloring replaces one alloca with another, transfer the SSPLayoutKind |
| 80 | // tag from the remapped to the target alloca. The remapped alloca should |
| 81 | // have a size smaller than or equal to the replacement alloca. |
| 82 | SSPLayoutMap::iterator I = Layout.find(From); |
| 83 | if (I != Layout.end()) { |
| 84 | SSPLayoutKind Kind = I->second; |
| 85 | Layout.erase(I); |
| 86 | |
| 87 | // Transfer the tag, but make sure that SSPLK_AddrOf does not overwrite |
| 88 | // SSPLK_SmallArray or SSPLK_LargeArray, and make sure that |
| 89 | // SSPLK_SmallArray does not overwrite SSPLK_LargeArray. |
| 90 | I = Layout.find(To); |
| 91 | if (I == Layout.end()) |
| 92 | Layout.insert(std::make_pair(To, Kind)); |
| 93 | else if (I->second != SSPLK_LargeArray && Kind != SSPLK_AddrOf) |
| 94 | I->second = Kind; |
| 95 | } |
| 96 | } |
| 97 | |
| Matthias Braun | e89461a | 2017-06-05 22:59:21 +0000 | [diff] [blame] | 98 | void StackProtector::getAnalysisUsage(AnalysisUsage &AU) const { |
| 99 | AU.addRequired<TargetPassConfig>(); |
| 100 | AU.addPreserved<DominatorTreeWrapperPass>(); |
| 101 | } |
| 102 | |
| Bill Wendling | 05d8417 | 2008-11-04 02:10:20 +0000 | [diff] [blame] | 103 | bool StackProtector::runOnFunction(Function &Fn) { |
| 104 | F = &Fn; |
| 105 | M = F->getParent(); |
| Chandler Carruth | 7352302 | 2014-01-13 13:07:17 +0000 | [diff] [blame] | 106 | DominatorTreeWrapperPass *DTWP = |
| 107 | getAnalysisIfAvailable<DominatorTreeWrapperPass>(); |
| Craig Topper | c0196b1 | 2014-04-14 00:51:57 +0000 | [diff] [blame] | 108 | DT = DTWP ? &DTWP->getDomTree() : nullptr; |
| Francis Visoiu Mistrih | 8b61764 | 2017-05-18 17:21:13 +0000 | [diff] [blame] | 109 | TM = &getAnalysis<TargetPassConfig>().getTM<TargetMachine>(); |
| 110 | Trip = TM->getTargetTriple(); |
| Eric Christopher | 3372620 | 2015-01-27 08:48:42 +0000 | [diff] [blame] | 111 | TLI = TM->getSubtargetImpl(Fn)->getTargetLowering(); |
| Tim Shen | 0012756 | 2016-04-08 21:26:31 +0000 | [diff] [blame] | 112 | HasPrologue = false; |
| 113 | HasIRCheck = false; |
| Bill Wendling | 05d8417 | 2008-11-04 02:10:20 +0000 | [diff] [blame] | 114 | |
| Duncan P. N. Exon Smith | 70eb9c5 | 2015-02-14 01:44:41 +0000 | [diff] [blame] | 115 | Attribute Attr = Fn.getFnAttribute("stack-protector-buffer-size"); |
| Renato Golin | e195f9c | 2014-01-21 10:24:35 +0000 | [diff] [blame] | 116 | if (Attr.isStringAttribute() && |
| 117 | Attr.getValueAsString().getAsInteger(10, SSPBufferSize)) |
| Etienne Bergeron | 22bfa83 | 2016-06-07 20:15:35 +0000 | [diff] [blame] | 118 | return false; // Invalid integer string |
| Bill Wendling | c02a0aa | 2013-07-22 20:15:21 +0000 | [diff] [blame] | 119 | |
| Josh Magee | adfde5f | 2014-04-17 19:08:36 +0000 | [diff] [blame] | 120 | if (!RequiresStackProtector()) |
| 121 | return false; |
| 122 | |
| Etienne Bergeron | 22bfa83 | 2016-06-07 20:15:35 +0000 | [diff] [blame] | 123 | // TODO(etienneb): Functions with funclets are not correctly supported now. |
| 124 | // Do nothing if this is funclet-based personality. |
| 125 | if (Fn.hasPersonalityFn()) { |
| 126 | EHPersonality Personality = classifyEHPersonality(Fn.getPersonalityFn()); |
| 127 | if (isFuncletEHPersonality(Personality)) |
| 128 | return false; |
| 129 | } |
| 130 | |
| Bill Wendling | 7c8f96a | 2013-01-23 06:43:53 +0000 | [diff] [blame] | 131 | ++NumFunProtected; |
| Bill Wendling | 782e834 | 2008-11-05 00:00:21 +0000 | [diff] [blame] | 132 | return InsertStackProtectors(); |
| Bill Wendling | 05d8417 | 2008-11-04 02:10:20 +0000 | [diff] [blame] | 133 | } |
| 134 | |
| Josh Magee | 3f1c0e3 | 2013-10-29 21:16:16 +0000 | [diff] [blame] | 135 | /// \param [out] IsLarge is set to true if a protectable array is found and |
| 136 | /// it is "large" ( >= ssp-buffer-size). In the case of a structure with |
| 137 | /// multiple arrays, this gets set if any of them is large. |
| 138 | bool StackProtector::ContainsProtectableArray(Type *Ty, bool &IsLarge, |
| Josh Magee | 7245f1d | 2013-10-30 02:25:14 +0000 | [diff] [blame] | 139 | bool Strong, |
| 140 | bool InStruct) const { |
| 141 | if (!Ty) |
| 142 | return false; |
| Bill Wendling | bfb9b75 | 2012-08-17 20:59:56 +0000 | [diff] [blame] | 143 | if (ArrayType *AT = dyn_cast<ArrayType>(Ty)) { |
| 144 | if (!AT->getElementType()->isIntegerTy(8)) { |
| Bill Wendling | bfb9b75 | 2012-08-17 20:59:56 +0000 | [diff] [blame] | 145 | // If we're on a non-Darwin platform or we're inside of a structure, don't |
| 146 | // add stack protectors unless the array is a character array. |
| Josh Magee | 3f1c0e3 | 2013-10-29 21:16:16 +0000 | [diff] [blame] | 147 | // However, in strong mode any array, regardless of type and size, |
| 148 | // triggers a protector. |
| 149 | if (!Strong && (InStruct || !Trip.isOSDarwin())) |
| 150 | return false; |
| Bill Wendling | bfb9b75 | 2012-08-17 20:59:56 +0000 | [diff] [blame] | 151 | } |
| 152 | |
| 153 | // If an array has more than SSPBufferSize bytes of allocated space, then we |
| 154 | // emit stack protectors. |
| Mehdi Amini | ed6edbf | 2015-07-07 23:38:49 +0000 | [diff] [blame] | 155 | if (SSPBufferSize <= M->getDataLayout().getTypeAllocSize(AT)) { |
| Josh Magee | 3f1c0e3 | 2013-10-29 21:16:16 +0000 | [diff] [blame] | 156 | IsLarge = true; |
| 157 | return true; |
| Josh Magee | 7245f1d | 2013-10-30 02:25:14 +0000 | [diff] [blame] | 158 | } |
| Josh Magee | 3f1c0e3 | 2013-10-29 21:16:16 +0000 | [diff] [blame] | 159 | |
| 160 | if (Strong) |
| 161 | // Require a protector for all arrays in strong mode |
| Bill Wendling | bfb9b75 | 2012-08-17 20:59:56 +0000 | [diff] [blame] | 162 | return true; |
| 163 | } |
| 164 | |
| 165 | const StructType *ST = dyn_cast<StructType>(Ty); |
| Josh Magee | 7245f1d | 2013-10-30 02:25:14 +0000 | [diff] [blame] | 166 | if (!ST) |
| 167 | return false; |
| Bill Wendling | bfb9b75 | 2012-08-17 20:59:56 +0000 | [diff] [blame] | 168 | |
| Josh Magee | 3f1c0e3 | 2013-10-29 21:16:16 +0000 | [diff] [blame] | 169 | bool NeedsProtector = false; |
| Bill Wendling | bfb9b75 | 2012-08-17 20:59:56 +0000 | [diff] [blame] | 170 | for (StructType::element_iterator I = ST->element_begin(), |
| Josh Magee | 7245f1d | 2013-10-30 02:25:14 +0000 | [diff] [blame] | 171 | E = ST->element_end(); |
| 172 | I != E; ++I) |
| Josh Magee | 3f1c0e3 | 2013-10-29 21:16:16 +0000 | [diff] [blame] | 173 | if (ContainsProtectableArray(*I, IsLarge, Strong, true)) { |
| 174 | // If the element is a protectable array and is large (>= SSPBufferSize) |
| 175 | // then we are done. If the protectable array is not large, then |
| 176 | // keep looking in case a subsequent element is a large array. |
| 177 | if (IsLarge) |
| 178 | return true; |
| 179 | NeedsProtector = true; |
| 180 | } |
| Bill Wendling | bfb9b75 | 2012-08-17 20:59:56 +0000 | [diff] [blame] | 181 | |
| Josh Magee | 3f1c0e3 | 2013-10-29 21:16:16 +0000 | [diff] [blame] | 182 | return NeedsProtector; |
| Bill Wendling | bfb9b75 | 2012-08-17 20:59:56 +0000 | [diff] [blame] | 183 | } |
| 184 | |
| Bill Wendling | 7c8f96a | 2013-01-23 06:43:53 +0000 | [diff] [blame] | 185 | bool StackProtector::HasAddressTaken(const Instruction *AI) { |
| Chandler Carruth | cdf4788 | 2014-03-09 03:16:01 +0000 | [diff] [blame] | 186 | for (const User *U : AI->users()) { |
| Bill Wendling | 7c8f96a | 2013-01-23 06:43:53 +0000 | [diff] [blame] | 187 | if (const StoreInst *SI = dyn_cast<StoreInst>(U)) { |
| 188 | if (AI == SI->getValueOperand()) |
| 189 | return true; |
| 190 | } else if (const PtrToIntInst *SI = dyn_cast<PtrToIntInst>(U)) { |
| 191 | if (AI == SI->getOperand(0)) |
| 192 | return true; |
| 193 | } else if (isa<CallInst>(U)) { |
| 194 | return true; |
| 195 | } else if (isa<InvokeInst>(U)) { |
| 196 | return true; |
| 197 | } else if (const SelectInst *SI = dyn_cast<SelectInst>(U)) { |
| 198 | if (HasAddressTaken(SI)) |
| 199 | return true; |
| 200 | } else if (const PHINode *PN = dyn_cast<PHINode>(U)) { |
| 201 | // Keep track of what PHI nodes we have already visited to ensure |
| 202 | // they are only visited once. |
| David Blaikie | 70573dc | 2014-11-19 07:49:26 +0000 | [diff] [blame] | 203 | if (VisitedPHIs.insert(PN).second) |
| Bill Wendling | 7c8f96a | 2013-01-23 06:43:53 +0000 | [diff] [blame] | 204 | if (HasAddressTaken(PN)) |
| 205 | return true; |
| 206 | } else if (const GetElementPtrInst *GEP = dyn_cast<GetElementPtrInst>(U)) { |
| 207 | if (HasAddressTaken(GEP)) |
| 208 | return true; |
| 209 | } else if (const BitCastInst *BI = dyn_cast<BitCastInst>(U)) { |
| 210 | if (HasAddressTaken(BI)) |
| 211 | return true; |
| 212 | } |
| 213 | } |
| 214 | return false; |
| 215 | } |
| 216 | |
| 217 | /// \brief Check whether or not this function needs a stack protector based |
| 218 | /// upon the stack protector level. |
| 219 | /// |
| 220 | /// We use two heuristics: a standard (ssp) and strong (sspstrong). |
| 221 | /// The standard heuristic which will add a guard variable to functions that |
| 222 | /// call alloca with a either a variable size or a size >= SSPBufferSize, |
| 223 | /// functions with character buffers larger than SSPBufferSize, and functions |
| 224 | /// with aggregates containing character buffers larger than SSPBufferSize. The |
| 225 | /// strong heuristic will add a guard variables to functions that call alloca |
| 226 | /// regardless of size, functions with any buffer regardless of type and size, |
| 227 | /// functions with aggregates that contain any buffer regardless of type and |
| 228 | /// size, and functions that contain stack-based variables that have had their |
| 229 | /// address taken. |
| 230 | bool StackProtector::RequiresStackProtector() { |
| 231 | bool Strong = false; |
| Josh Magee | 3f1c0e3 | 2013-10-29 21:16:16 +0000 | [diff] [blame] | 232 | bool NeedsProtector = false; |
| Tim Shen | 0012756 | 2016-04-08 21:26:31 +0000 | [diff] [blame] | 233 | for (const BasicBlock &BB : *F) |
| 234 | for (const Instruction &I : BB) |
| 235 | if (const CallInst *CI = dyn_cast<CallInst>(&I)) |
| 236 | if (CI->getCalledFunction() == |
| 237 | Intrinsic::getDeclaration(F->getParent(), |
| 238 | Intrinsic::stackprotector)) |
| 239 | HasPrologue = true; |
| 240 | |
| Evgeniy Stepanov | f17120a | 2016-04-11 22:27:48 +0000 | [diff] [blame] | 241 | if (F->hasFnAttribute(Attribute::SafeStack)) |
| 242 | return false; |
| 243 | |
| David Bozier | 5159968 | 2017-02-28 16:02:37 +0000 | [diff] [blame] | 244 | // We are constructing the OptimizationRemarkEmitter on the fly rather than |
| 245 | // using the analysis pass to avoid building DominatorTree and LoopInfo which |
| 246 | // are not available this late in the IR pipeline. |
| 247 | OptimizationRemarkEmitter ORE(F); |
| David Bozier | 5159968 | 2017-02-28 16:02:37 +0000 | [diff] [blame] | 248 | |
| Duncan P. N. Exon Smith | 70eb9c5 | 2015-02-14 01:44:41 +0000 | [diff] [blame] | 249 | if (F->hasFnAttribute(Attribute::StackProtectReq)) { |
| Vivek Pandya | 9590658 | 2017-10-11 17:12:59 +0000 | [diff] [blame] | 250 | ORE.emit([&]() { |
| 251 | return OptimizationRemark(DEBUG_TYPE, "StackProtectorRequested", F) |
| Adam Nemet | 5361b82 | 2017-03-09 06:10:27 +0000 | [diff] [blame] | 252 | << "Stack protection applied to function " |
| 253 | << ore::NV("Function", F) |
| Vivek Pandya | 9590658 | 2017-10-11 17:12:59 +0000 | [diff] [blame] | 254 | << " due to a function attribute or command-line switch"; |
| 255 | }); |
| Josh Magee | 3f1c0e3 | 2013-10-29 21:16:16 +0000 | [diff] [blame] | 256 | NeedsProtector = true; |
| 257 | Strong = true; // Use the same heuristic as strong to determine SSPLayout |
| Duncan P. N. Exon Smith | 70eb9c5 | 2015-02-14 01:44:41 +0000 | [diff] [blame] | 258 | } else if (F->hasFnAttribute(Attribute::StackProtectStrong)) |
| Bill Wendling | 7c8f96a | 2013-01-23 06:43:53 +0000 | [diff] [blame] | 259 | Strong = true; |
| Tim Shen | 0012756 | 2016-04-08 21:26:31 +0000 | [diff] [blame] | 260 | else if (HasPrologue) |
| 261 | NeedsProtector = true; |
| Duncan P. N. Exon Smith | 70eb9c5 | 2015-02-14 01:44:41 +0000 | [diff] [blame] | 262 | else if (!F->hasFnAttribute(Attribute::StackProtect)) |
| Bill Wendling | eeb0415 | 2008-11-18 05:32:11 +0000 | [diff] [blame] | 263 | return false; |
| 264 | |
| Saleem Abdulrasool | 57b5fe5 | 2014-12-20 21:37:51 +0000 | [diff] [blame] | 265 | for (const BasicBlock &BB : *F) { |
| 266 | for (const Instruction &I : BB) { |
| 267 | if (const AllocaInst *AI = dyn_cast<AllocaInst>(&I)) { |
| Bill Wendling | 7c8f96a | 2013-01-23 06:43:53 +0000 | [diff] [blame] | 268 | if (AI->isArrayAllocation()) { |
| Vivek Pandya | 9590658 | 2017-10-11 17:12:59 +0000 | [diff] [blame] | 269 | auto RemarkBuilder = [&]() { |
| 270 | return OptimizationRemark(DEBUG_TYPE, "StackProtectorAllocaOrArray", |
| 271 | &I) |
| 272 | << "Stack protection applied to function " |
| 273 | << ore::NV("Function", F) |
| 274 | << " due to a call to alloca or use of a variable length " |
| 275 | "array"; |
| 276 | }; |
| Saleem Abdulrasool | 57b5fe5 | 2014-12-20 21:37:51 +0000 | [diff] [blame] | 277 | if (const auto *CI = dyn_cast<ConstantInt>(AI->getArraySize())) { |
| Josh Magee | 3f1c0e3 | 2013-10-29 21:16:16 +0000 | [diff] [blame] | 278 | if (CI->getLimitedValue(SSPBufferSize) >= SSPBufferSize) { |
| Bill Wendling | 7c8f96a | 2013-01-23 06:43:53 +0000 | [diff] [blame] | 279 | // A call to alloca with size >= SSPBufferSize requires |
| 280 | // stack protectors. |
| Josh Magee | 3f1c0e3 | 2013-10-29 21:16:16 +0000 | [diff] [blame] | 281 | Layout.insert(std::make_pair(AI, SSPLK_LargeArray)); |
| Vivek Pandya | 9590658 | 2017-10-11 17:12:59 +0000 | [diff] [blame] | 282 | ORE.emit(RemarkBuilder); |
| Josh Magee | 3f1c0e3 | 2013-10-29 21:16:16 +0000 | [diff] [blame] | 283 | NeedsProtector = true; |
| 284 | } else if (Strong) { |
| 285 | // Require protectors for all alloca calls in strong mode. |
| 286 | Layout.insert(std::make_pair(AI, SSPLK_SmallArray)); |
| Vivek Pandya | 9590658 | 2017-10-11 17:12:59 +0000 | [diff] [blame] | 287 | ORE.emit(RemarkBuilder); |
| Josh Magee | 3f1c0e3 | 2013-10-29 21:16:16 +0000 | [diff] [blame] | 288 | NeedsProtector = true; |
| 289 | } |
| Bill Wendling | c02a0aa | 2013-07-22 20:15:21 +0000 | [diff] [blame] | 290 | } else { |
| 291 | // A call to alloca with a variable size requires protectors. |
| Josh Magee | 3f1c0e3 | 2013-10-29 21:16:16 +0000 | [diff] [blame] | 292 | Layout.insert(std::make_pair(AI, SSPLK_LargeArray)); |
| Vivek Pandya | 9590658 | 2017-10-11 17:12:59 +0000 | [diff] [blame] | 293 | ORE.emit(RemarkBuilder); |
| Josh Magee | 3f1c0e3 | 2013-10-29 21:16:16 +0000 | [diff] [blame] | 294 | NeedsProtector = true; |
| Bill Wendling | c02a0aa | 2013-07-22 20:15:21 +0000 | [diff] [blame] | 295 | } |
| Josh Magee | 3f1c0e3 | 2013-10-29 21:16:16 +0000 | [diff] [blame] | 296 | continue; |
| Bill Wendling | 7c8f96a | 2013-01-23 06:43:53 +0000 | [diff] [blame] | 297 | } |
| 298 | |
| Josh Magee | 3f1c0e3 | 2013-10-29 21:16:16 +0000 | [diff] [blame] | 299 | bool IsLarge = false; |
| 300 | if (ContainsProtectableArray(AI->getAllocatedType(), IsLarge, Strong)) { |
| 301 | Layout.insert(std::make_pair(AI, IsLarge ? SSPLK_LargeArray |
| 302 | : SSPLK_SmallArray)); |
| Vivek Pandya | 9590658 | 2017-10-11 17:12:59 +0000 | [diff] [blame] | 303 | ORE.emit([&]() { |
| 304 | return OptimizationRemark(DEBUG_TYPE, "StackProtectorBuffer", &I) |
| Adam Nemet | 5361b82 | 2017-03-09 06:10:27 +0000 | [diff] [blame] | 305 | << "Stack protection applied to function " |
| 306 | << ore::NV("Function", F) |
| 307 | << " due to a stack allocated buffer or struct containing a " |
| Vivek Pandya | 9590658 | 2017-10-11 17:12:59 +0000 | [diff] [blame] | 308 | "buffer"; |
| 309 | }); |
| Josh Magee | 3f1c0e3 | 2013-10-29 21:16:16 +0000 | [diff] [blame] | 310 | NeedsProtector = true; |
| 311 | continue; |
| 312 | } |
| Bill Wendling | eeb0415 | 2008-11-18 05:32:11 +0000 | [diff] [blame] | 313 | |
| Bill Wendling | 7c8f96a | 2013-01-23 06:43:53 +0000 | [diff] [blame] | 314 | if (Strong && HasAddressTaken(AI)) { |
| Michael Gottesman | 62c5d71 | 2013-08-20 08:46:16 +0000 | [diff] [blame] | 315 | ++NumAddrTaken; |
| Josh Magee | 3f1c0e3 | 2013-10-29 21:16:16 +0000 | [diff] [blame] | 316 | Layout.insert(std::make_pair(AI, SSPLK_AddrOf)); |
| Vivek Pandya | 9590658 | 2017-10-11 17:12:59 +0000 | [diff] [blame] | 317 | ORE.emit([&]() { |
| 318 | return OptimizationRemark(DEBUG_TYPE, "StackProtectorAddressTaken", |
| 319 | &I) |
| 320 | << "Stack protection applied to function " |
| 321 | << ore::NV("Function", F) |
| 322 | << " due to the address of a local variable being taken"; |
| 323 | }); |
| Josh Magee | 3f1c0e3 | 2013-10-29 21:16:16 +0000 | [diff] [blame] | 324 | NeedsProtector = true; |
| Bill Wendling | 7c8f96a | 2013-01-23 06:43:53 +0000 | [diff] [blame] | 325 | } |
| Bill Wendling | eeb0415 | 2008-11-18 05:32:11 +0000 | [diff] [blame] | 326 | } |
| Bill Wendling | 7c8f96a | 2013-01-23 06:43:53 +0000 | [diff] [blame] | 327 | } |
| Bill Wendling | eeb0415 | 2008-11-18 05:32:11 +0000 | [diff] [blame] | 328 | } |
| 329 | |
| Josh Magee | 3f1c0e3 | 2013-10-29 21:16:16 +0000 | [diff] [blame] | 330 | return NeedsProtector; |
| Bill Wendling | eeb0415 | 2008-11-18 05:32:11 +0000 | [diff] [blame] | 331 | } |
| 332 | |
| Tim Shen | e885d5e | 2016-04-19 19:40:37 +0000 | [diff] [blame] | 333 | /// Create a stack guard loading and populate whether SelectionDAG SSP is |
| 334 | /// supported. |
| 335 | static Value *getStackGuard(const TargetLoweringBase *TLI, Module *M, |
| 336 | IRBuilder<> &B, |
| 337 | bool *SupportsSelectionDAGSP = nullptr) { |
| 338 | if (Value *Guard = TLI->getIRStackGuard(B)) |
| 339 | return B.CreateLoad(Guard, true, "StackGuard"); |
| 340 | |
| 341 | // Use SelectionDAG SSP handling, since there isn't an IR guard. |
| 342 | // |
| 343 | // This is more or less weird, since we optionally output whether we |
| 344 | // should perform a SelectionDAG SP here. The reason is that it's strictly |
| 345 | // defined as !TLI->getIRStackGuard(B), where getIRStackGuard is also |
| 346 | // mutating. There is no way to get this bit without mutating the IR, so |
| 347 | // getting this bit has to happen in this right time. |
| 348 | // |
| 349 | // We could have define a new function TLI::supportsSelectionDAGSP(), but that |
| 350 | // will put more burden on the backends' overriding work, especially when it |
| 351 | // actually conveys the same information getIRStackGuard() already gives. |
| 352 | if (SupportsSelectionDAGSP) |
| 353 | *SupportsSelectionDAGSP = true; |
| 354 | TLI->insertSSPDeclarations(*M); |
| 355 | return B.CreateCall(Intrinsic::getDeclaration(M, Intrinsic::stackguard)); |
| 356 | } |
| 357 | |
| 358 | /// Insert code into the entry block that stores the stack guard |
| Michael Gottesman | a6188f9 | 2013-07-22 20:44:11 +0000 | [diff] [blame] | 359 | /// variable onto the stack: |
| 360 | /// |
| 361 | /// entry: |
| 362 | /// StackGuardSlot = alloca i8* |
| Tim Shen | e885d5e | 2016-04-19 19:40:37 +0000 | [diff] [blame] | 363 | /// StackGuard = <stack guard> |
| 364 | /// call void @llvm.stackprotector(StackGuard, StackGuardSlot) |
| Michael Gottesman | a6188f9 | 2013-07-22 20:44:11 +0000 | [diff] [blame] | 365 | /// |
| Michael Gottesman | 5e57068 | 2013-08-20 08:36:53 +0000 | [diff] [blame] | 366 | /// Returns true if the platform/triple supports the stackprotectorcreate pseudo |
| 367 | /// node. |
| 368 | static bool CreatePrologue(Function *F, Module *M, ReturnInst *RI, |
| Tim Shen | e885d5e | 2016-04-19 19:40:37 +0000 | [diff] [blame] | 369 | const TargetLoweringBase *TLI, AllocaInst *&AI) { |
| Michael Gottesman | 5e57068 | 2013-08-20 08:36:53 +0000 | [diff] [blame] | 370 | bool SupportsSelectionDAGSP = false; |
| Evgeniy Stepanov | dde29e2 | 2016-04-05 22:41:50 +0000 | [diff] [blame] | 371 | IRBuilder<> B(&F->getEntryBlock().front()); |
| Tim Shen | 0012756 | 2016-04-08 21:26:31 +0000 | [diff] [blame] | 372 | PointerType *PtrTy = Type::getInt8PtrTy(RI->getContext()); |
| Craig Topper | c0196b1 | 2014-04-14 00:51:57 +0000 | [diff] [blame] | 373 | AI = B.CreateAlloca(PtrTy, nullptr, "StackGuardSlot"); |
| Tim Shen | e885d5e | 2016-04-19 19:40:37 +0000 | [diff] [blame] | 374 | |
| Etienne Bergeron | 22bfa83 | 2016-06-07 20:15:35 +0000 | [diff] [blame] | 375 | Value *GuardSlot = getStackGuard(TLI, M, B, &SupportsSelectionDAGSP); |
| David Blaikie | ff6409d | 2015-05-18 22:13:54 +0000 | [diff] [blame] | 376 | B.CreateCall(Intrinsic::getDeclaration(M, Intrinsic::stackprotector), |
| Etienne Bergeron | 22bfa83 | 2016-06-07 20:15:35 +0000 | [diff] [blame] | 377 | {GuardSlot, AI}); |
| Michael Gottesman | 5e57068 | 2013-08-20 08:36:53 +0000 | [diff] [blame] | 378 | return SupportsSelectionDAGSP; |
| Michael Gottesman | a6188f9 | 2013-07-22 20:44:11 +0000 | [diff] [blame] | 379 | } |
| 380 | |
| Bill Wendling | 782e834 | 2008-11-05 00:00:21 +0000 | [diff] [blame] | 381 | /// InsertStackProtectors - Insert code into the prologue and epilogue of the |
| 382 | /// function. |
| 383 | /// |
| 384 | /// - The prologue code loads and stores the stack guard onto the stack. |
| 385 | /// - The epilogue checks the value stored in the prologue against the original |
| 386 | /// value. It calls __stack_chk_fail if they differ. |
| 387 | bool StackProtector::InsertStackProtectors() { |
| Michael Gottesman | 76c44be | 2013-08-20 08:56:26 +0000 | [diff] [blame] | 388 | bool SupportsSelectionDAGSP = |
| Josh Magee | 7245f1d | 2013-10-30 02:25:14 +0000 | [diff] [blame] | 389 | EnableSelectionDAGSP && !TM->Options.EnableFastISel; |
| Craig Topper | c0196b1 | 2014-04-14 00:51:57 +0000 | [diff] [blame] | 390 | AllocaInst *AI = nullptr; // Place on stack that stores the stack guard. |
| Bill Wendling | eb4268d | 2008-11-07 01:23:58 +0000 | [diff] [blame] | 391 | |
| Josh Magee | 7245f1d | 2013-10-30 02:25:14 +0000 | [diff] [blame] | 392 | for (Function::iterator I = F->begin(), E = F->end(); I != E;) { |
| Duncan P. N. Exon Smith | f1ff53e | 2015-10-09 22:56:24 +0000 | [diff] [blame] | 393 | BasicBlock *BB = &*I++; |
| Bill Wendling | eeb0415 | 2008-11-18 05:32:11 +0000 | [diff] [blame] | 394 | ReturnInst *RI = dyn_cast<ReturnInst>(BB->getTerminator()); |
| Michael Gottesman | dc985ef | 2013-08-20 08:56:28 +0000 | [diff] [blame] | 395 | if (!RI) |
| 396 | continue; |
| Bill Wendling | eb4268d | 2008-11-07 01:23:58 +0000 | [diff] [blame] | 397 | |
| Etienne Bergeron | 22bfa83 | 2016-06-07 20:15:35 +0000 | [diff] [blame] | 398 | // Generate prologue instrumentation if not already generated. |
| Michael Gottesman | 8afcf3a | 2013-08-09 21:26:18 +0000 | [diff] [blame] | 399 | if (!HasPrologue) { |
| 400 | HasPrologue = true; |
| Tim Shen | e885d5e | 2016-04-19 19:40:37 +0000 | [diff] [blame] | 401 | SupportsSelectionDAGSP &= CreatePrologue(F, M, RI, TLI, AI); |
| Michael Gottesman | 62c5d71 | 2013-08-20 08:46:16 +0000 | [diff] [blame] | 402 | } |
| Michael Gottesman | 5e57068 | 2013-08-20 08:36:53 +0000 | [diff] [blame] | 403 | |
| Etienne Bergeron | 22bfa83 | 2016-06-07 20:15:35 +0000 | [diff] [blame] | 404 | // SelectionDAG based code generation. Nothing else needs to be done here. |
| 405 | // The epilogue instrumentation is postponed to SelectionDAG. |
| 406 | if (SupportsSelectionDAGSP) |
| 407 | break; |
| 408 | |
| 409 | // Set HasIRCheck to true, so that SelectionDAG will not generate its own |
| 410 | // version. SelectionDAG called 'shouldEmitSDCheck' to check whether |
| 411 | // instrumentation has already been generated. |
| 412 | HasIRCheck = true; |
| 413 | |
| 414 | // Generate epilogue instrumentation. The epilogue intrumentation can be |
| 415 | // function-based or inlined depending on which mechanism the target is |
| 416 | // providing. |
| 417 | if (Value* GuardCheck = TLI->getSSPStackGuardCheck(*M)) { |
| 418 | // Generate the function-based epilogue instrumentation. |
| 419 | // The target provides a guard check function, generate a call to it. |
| 420 | IRBuilder<> B(RI); |
| 421 | LoadInst *Guard = B.CreateLoad(AI, true, "Guard"); |
| 422 | CallInst *Call = B.CreateCall(GuardCheck, {Guard}); |
| 423 | llvm::Function *Function = cast<llvm::Function>(GuardCheck); |
| 424 | Call->setAttributes(Function->getAttributes()); |
| 425 | Call->setCallingConv(Function->getCallingConv()); |
| 426 | } else { |
| 427 | // Generate the epilogue with inline instrumentation. |
| Michael Gottesman | 56e246b | 2013-08-20 08:46:13 +0000 | [diff] [blame] | 428 | // If we do not support SelectionDAG based tail calls, generate IR level |
| 429 | // tail calls. |
| 430 | // |
| Michael Gottesman | 5e57068 | 2013-08-20 08:36:53 +0000 | [diff] [blame] | 431 | // For each block with a return instruction, convert this: |
| 432 | // |
| 433 | // return: |
| 434 | // ... |
| 435 | // ret ... |
| 436 | // |
| 437 | // into this: |
| 438 | // |
| 439 | // return: |
| 440 | // ... |
| Tim Shen | e885d5e | 2016-04-19 19:40:37 +0000 | [diff] [blame] | 441 | // %1 = <stack guard> |
| Michael Gottesman | 5e57068 | 2013-08-20 08:36:53 +0000 | [diff] [blame] | 442 | // %2 = load StackGuardSlot |
| 443 | // %3 = cmp i1 %1, %2 |
| 444 | // br i1 %3, label %SP_return, label %CallStackCheckFailBlk |
| 445 | // |
| 446 | // SP_return: |
| 447 | // ret ... |
| 448 | // |
| 449 | // CallStackCheckFailBlk: |
| 450 | // call void @__stack_chk_fail() |
| 451 | // unreachable |
| 452 | |
| 453 | // Create the FailBB. We duplicate the BB every time since the MI tail |
| 454 | // merge pass will merge together all of the various BB into one including |
| Michael Gottesman | 62c5d71 | 2013-08-20 08:46:16 +0000 | [diff] [blame] | 455 | // fail BB generated by the stack protector pseudo instruction. |
| Michael Gottesman | 5e57068 | 2013-08-20 08:36:53 +0000 | [diff] [blame] | 456 | BasicBlock *FailBB = CreateFailBB(); |
| Michael Gottesman | 62c5d71 | 2013-08-20 08:46:16 +0000 | [diff] [blame] | 457 | |
| Michael Gottesman | 5e57068 | 2013-08-20 08:36:53 +0000 | [diff] [blame] | 458 | // Split the basic block before the return instruction. |
| Duncan P. N. Exon Smith | f1ff53e | 2015-10-09 22:56:24 +0000 | [diff] [blame] | 459 | BasicBlock *NewBB = BB->splitBasicBlock(RI->getIterator(), "SP_return"); |
| Michael Gottesman | 62c5d71 | 2013-08-20 08:46:16 +0000 | [diff] [blame] | 460 | |
| Michael Gottesman | 5e57068 | 2013-08-20 08:36:53 +0000 | [diff] [blame] | 461 | // Update the dominator tree if we need to. |
| 462 | if (DT && DT->isReachableFromEntry(BB)) { |
| 463 | DT->addNewBlock(NewBB, BB); |
| 464 | DT->addNewBlock(FailBB, BB); |
| 465 | } |
| Michael Gottesman | 62c5d71 | 2013-08-20 08:46:16 +0000 | [diff] [blame] | 466 | |
| Michael Gottesman | 5e57068 | 2013-08-20 08:36:53 +0000 | [diff] [blame] | 467 | // Remove default branch instruction to the new BB. |
| 468 | BB->getTerminator()->eraseFromParent(); |
| Michael Gottesman | 62c5d71 | 2013-08-20 08:46:16 +0000 | [diff] [blame] | 469 | |
| Michael Gottesman | 5e57068 | 2013-08-20 08:36:53 +0000 | [diff] [blame] | 470 | // Move the newly created basic block to the point right after the old |
| 471 | // basic block so that it's in the "fall through" position. |
| 472 | NewBB->moveAfter(BB); |
| Michael Gottesman | 62c5d71 | 2013-08-20 08:46:16 +0000 | [diff] [blame] | 473 | |
| Michael Gottesman | 5e57068 | 2013-08-20 08:36:53 +0000 | [diff] [blame] | 474 | // Generate the stack protector instructions in the old basic block. |
| Benjamin Kramer | d93817f | 2013-09-09 17:38:01 +0000 | [diff] [blame] | 475 | IRBuilder<> B(BB); |
| Tim Shen | e885d5e | 2016-04-19 19:40:37 +0000 | [diff] [blame] | 476 | Value *Guard = getStackGuard(TLI, M, B); |
| 477 | LoadInst *LI2 = B.CreateLoad(AI, true); |
| 478 | Value *Cmp = B.CreateICmpEQ(Guard, LI2); |
| Cong Hou | e93b8e1 | 2015-12-22 18:56:14 +0000 | [diff] [blame] | 479 | auto SuccessProb = |
| 480 | BranchProbabilityInfo::getBranchProbStackProtector(true); |
| 481 | auto FailureProb = |
| 482 | BranchProbabilityInfo::getBranchProbStackProtector(false); |
| Akira Hatanaka | b9991a2 | 2014-12-01 04:27:03 +0000 | [diff] [blame] | 483 | MDNode *Weights = MDBuilder(F->getContext()) |
| Cong Hou | e93b8e1 | 2015-12-22 18:56:14 +0000 | [diff] [blame] | 484 | .createBranchWeights(SuccessProb.getNumerator(), |
| 485 | FailureProb.getNumerator()); |
| Akira Hatanaka | b9991a2 | 2014-12-01 04:27:03 +0000 | [diff] [blame] | 486 | B.CreateCondBr(Cmp, NewBB, FailBB, Weights); |
| Bill Wendling | a0826e18 | 2008-11-06 23:55:49 +0000 | [diff] [blame] | 487 | } |
| Bill Wendling | 05d8417 | 2008-11-04 02:10:20 +0000 | [diff] [blame] | 488 | } |
| Bill Wendling | 782e834 | 2008-11-05 00:00:21 +0000 | [diff] [blame] | 489 | |
| Saleem Abdulrasool | 90c224a | 2014-12-21 21:52:38 +0000 | [diff] [blame] | 490 | // Return if we didn't modify any basic blocks. i.e., there are no return |
| Bill Wendling | a0826e18 | 2008-11-06 23:55:49 +0000 | [diff] [blame] | 491 | // statements in the function. |
| Rafael Espindola | 84921b9 | 2015-10-24 23:11:13 +0000 | [diff] [blame] | 492 | return HasPrologue; |
| Bill Wendling | 05d8417 | 2008-11-04 02:10:20 +0000 | [diff] [blame] | 493 | } |
| 494 | |
| 495 | /// CreateFailBB - Create a basic block to jump to when the stack protector |
| 496 | /// check fails. |
| Bill Wendling | 782e834 | 2008-11-05 00:00:21 +0000 | [diff] [blame] | 497 | BasicBlock *StackProtector::CreateFailBB() { |
| Rafael Espindola | aad6c24 | 2013-06-07 16:35:57 +0000 | [diff] [blame] | 498 | LLVMContext &Context = F->getContext(); |
| 499 | BasicBlock *FailBB = BasicBlock::Create(Context, "CallStackCheckFailBlk", F); |
| Benjamin Kramer | d93817f | 2013-09-09 17:38:01 +0000 | [diff] [blame] | 500 | IRBuilder<> B(FailBB); |
| Yunzhong Gao | b386955 | 2016-06-30 18:49:04 +0000 | [diff] [blame] | 501 | B.SetCurrentDebugLocation(DebugLoc::get(0, 0, F->getSubprogram())); |
| Simon Pilgrim | 2bfd912 | 2014-11-29 19:18:21 +0000 | [diff] [blame] | 502 | if (Trip.isOSOpenBSD()) { |
| Mehdi Amini | db11fdf | 2017-04-06 20:23:57 +0000 | [diff] [blame] | 503 | Constant *StackChkFail = |
| 504 | M->getOrInsertFunction("__stack_smash_handler", |
| 505 | Type::getVoidTy(Context), |
| Serge Guelton | 59a2d7b | 2017-04-11 15:01:18 +0000 | [diff] [blame] | 506 | Type::getInt8PtrTy(Context)); |
| Rafael Espindola | aad6c24 | 2013-06-07 16:35:57 +0000 | [diff] [blame] | 507 | |
| Benjamin Kramer | d93817f | 2013-09-09 17:38:01 +0000 | [diff] [blame] | 508 | B.CreateCall(StackChkFail, B.CreateGlobalStringPtr(F->getName(), "SSH")); |
| Rafael Espindola | aad6c24 | 2013-06-07 16:35:57 +0000 | [diff] [blame] | 509 | } else { |
| Saleem Abdulrasool | 90c224a | 2014-12-21 21:52:38 +0000 | [diff] [blame] | 510 | Constant *StackChkFail = |
| Serge Guelton | 59a2d7b | 2017-04-11 15:01:18 +0000 | [diff] [blame] | 511 | M->getOrInsertFunction("__stack_chk_fail", Type::getVoidTy(Context)); |
| 512 | |
| David Blaikie | ff6409d | 2015-05-18 22:13:54 +0000 | [diff] [blame] | 513 | B.CreateCall(StackChkFail, {}); |
| Rafael Espindola | aad6c24 | 2013-06-07 16:35:57 +0000 | [diff] [blame] | 514 | } |
| Benjamin Kramer | d93817f | 2013-09-09 17:38:01 +0000 | [diff] [blame] | 515 | B.CreateUnreachable(); |
| Bill Wendling | 782e834 | 2008-11-05 00:00:21 +0000 | [diff] [blame] | 516 | return FailBB; |
| Bill Wendling | 05d8417 | 2008-11-04 02:10:20 +0000 | [diff] [blame] | 517 | } |
| Tim Shen | 0012756 | 2016-04-08 21:26:31 +0000 | [diff] [blame] | 518 | |
| 519 | bool StackProtector::shouldEmitSDCheck(const BasicBlock &BB) const { |
| 520 | return HasPrologue && !HasIRCheck && dyn_cast<ReturnInst>(BB.getTerminator()); |
| 521 | } |