[PATCH] D17736: [SSP] Remove llvm.stackprotectorcheck.

[Evgeniy Stepanov via llvm-commits]

eugenis added a comment.

In http://reviews.llvm.org/D17736#395796, @timshen wrote:

> Hi eugenis, I changed the function name getStackCookieLocation to getIRStackGuard, since there are two SSP code paths and such a name seems to cause less confusion. Do you think it's a fine name?


Absolutely.


================
Comment at: include/llvm/Target/TargetLowering.h:1016
@@ -1015,4 +1015,3 @@
   /// returns the address of that location. Otherwise, returns nullptr.
-  virtual Value *getStackCookieLocation(IRBuilder<> &IRB) const {
-    return nullptr;
-  }
+  /// Returning nullptr means to use SelectionDAG SSP handling.
+  virtual Value *getIRStackGuard(IRBuilder<> &IRB) const;
----------------
This comment makes it sound like this function makes the decision to use SDAG vs IR stack protector.



http://reviews.llvm.org/D17736