From: Michael Gottesman <mgottesman@apple.com>
Date: Tue, 20 Aug 2013 08:56:23 +0000 (+0000)
Subject: [stackprotector] Added significantly longer comment to FindPotentialTailCall to make... 
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=b99272a521ecffe8d021306713bd51fafc85721e;p=oota-llvm.git

[stackprotector] Added significantly longer comment to FindPotentialTailCall to make clear its relationship to llvm::isInTailCallPosition.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@188770 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/CodeGen/StackProtector.cpp b/lib/CodeGen/StackProtector.cpp
index b45459a4f6d..2cd2219ac92 100644
--- a/lib/CodeGen/StackProtector.cpp
+++ b/lib/CodeGen/StackProtector.cpp
@@ -280,7 +280,12 @@ static bool InstructionWillNotHaveChain(const Instruction *I) {
 /// Identify if RI has a previous instruction in the "Tail Position" and return
 /// it. Otherwise return 0.
 ///
-/// This is based off of the code in llvm::isInTailCallPosition
+/// This is based off of the code in llvm::isInTailCallPosition. The difference
+/// is that it inverts the first part of llvm::isInTailCallPosition since
+/// isInTailCallPosition is checking if a call is in a tail call position, and
+/// we are searching for an unknown tail call that might be in the tail call
+/// position. Once we find the call though, the code uses the same refactored
+/// code, returnTypeIsEligibleForTailCall.
 static CallInst *FindPotentialTailCall(BasicBlock *BB, ReturnInst *RI,
                                        const TargetLoweringBase *TLI) {
   // Establish a reasonable upper bound on the maximum amount of instructions we