From abd6f28a74e74303725ec123bdfe9202617115c8 Mon Sep 17 00:00:00 2001 From: Gabor Greif Date: Sun, 1 Mar 2009 16:38:10 +0000 Subject: Reuse a technique (pioneered for BasicBlocks) of superposing ilist with its sentinel. This is quite a win when a function really has a basic block. When the function is just a declaration (and stays so) the old way did not allocate a sentinel. So this change is most beneficial when the ratio of function definition to declaration is high. I.e. linkers etc. Incidentally these are the most resource demanding applications, so I expect that the reduced malloc traffic, locality and space savings outweigh the cost of addition of two pointers to Function. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@65776 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/VMCore/Function.cpp | 7 ------- 1 file changed, 7 deletions(-) (limited to 'lib/VMCore') diff --git a/lib/VMCore/Function.cpp b/lib/VMCore/Function.cpp index cff4457..fefe082 100644 --- a/lib/VMCore/Function.cpp +++ b/lib/VMCore/Function.cpp @@ -22,13 +22,6 @@ #include "llvm/ADT/StringExtras.h" using namespace llvm; -BasicBlock *ilist_traits::createSentinel() { - BasicBlock *Ret = BasicBlock::Create(); - // This should not be garbage monitored. - LeakDetector::removeGarbageObject(Ret); - return Ret; -} - iplist &ilist_traits::getList(Function *F) { return F->getBasicBlockList(); } -- cgit v1.1