[llvm-branch-commits] [cfe-branch] r119647 - /cfe/branches/Apple/whitney/tools/libclang/CIndex.cpp

Daniel Dunbar daniel at zuster.org
Wed Nov 17 18:37:25 PST 2010


Author: ddunbar
Date: Wed Nov 17 20:37:25 2010
New Revision: 119647

URL: http://llvm.org/viewvc/llvm-project?rev=119647&view=rev
Log:
Merge r119281:
--
Author: Ted Kremenek <kremenek at apple.com>
Date:   Mon Nov 15 22:23:24 2010 +0000

    Annotate CursorVisitor::VisitDataRecursive() with attribute 'noinline'.
    Clang currently uses a ridiculous amount of stack space when inlining
    this function, which can lead to premature stack overflows.

Modified:
    cfe/branches/Apple/whitney/tools/libclang/CIndex.cpp

Modified: cfe/branches/Apple/whitney/tools/libclang/CIndex.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/branches/Apple/whitney/tools/libclang/CIndex.cpp?rev=119647&r1=119646&r2=119647&view=diff
==============================================================================
--- cfe/branches/Apple/whitney/tools/libclang/CIndex.cpp (original)
+++ cfe/branches/Apple/whitney/tools/libclang/CIndex.cpp Wed Nov 17 20:37:25 2010
@@ -342,7 +342,7 @@
   bool IsInRegionOfInterest(CXCursor C);
   bool RunVisitorWorkList(VisitorWorkList &WL);
   void EnqueueWorkList(VisitorWorkList &WL, Stmt *S);
-  bool VisitDataRecursive(Stmt *S);
+  bool VisitDataRecursive(Stmt *S) __attribute__((noinline));
 };
 
 } // end anonymous namespace





More information about the llvm-branch-commits mailing list