[PATCH] D13789: [safestack] Fast access to the unsafe stack pointer on AArch64/Android (2nd try).

Evgeniy Stepanov via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 20 12:14:31 PDT 2015


eugenis added inline comments.

================
Comment at: lib/Transforms/Instrumentation/SafeStack.cpp:483
@@ -534,1 +482,3 @@
 
+Value *SafeStack::getOrCreateUnsafeStackPtr(IRBuilder<> &IRB, Function &F) {
+  // Check if there is a target-specific location for the unsafe stack pointer.
----------------
rengolin wrote:
> Moving this around makes it harder to review. Can you move it back where it was?
Sorry, did not noticed the method was moved. Fixed.


Repository:
  rL LLVM

http://reviews.llvm.org/D13789





More information about the llvm-commits mailing list