[cfe-commits] r128081 - /cfe/trunk/include/clang/Sema/Overload.h

Benjamin Kramer benny.kra at googlemail.com
Tue Mar 22 02:46:54 PDT 2011


Author: d0k
Date: Tue Mar 22 04:46:54 2011
New Revision: 128081

URL: http://llvm.org/viewvc/llvm-project?rev=128081&view=rev
Log:
Don't clear OverloadCandidateSets on destruction, it has no effect anymore.

And clearing a SmallPtrSet can be an expensive operation.

Modified:
    cfe/trunk/include/clang/Sema/Overload.h

Modified: cfe/trunk/include/clang/Sema/Overload.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Sema/Overload.h?rev=128081&r1=128080&r2=128081&view=diff
==============================================================================
--- cfe/trunk/include/clang/Sema/Overload.h (original)
+++ cfe/trunk/include/clang/Sema/Overload.h Tue Mar 22 04:46:54 2011
@@ -647,8 +647,6 @@
 
     /// \brief Clear out all of the candidates.
     void clear();
-    
-    ~OverloadCandidateSet() { clear(); }
 
     /// Find the best viable function on this overload set, if it exists.
     OverloadingResult BestViableFunction(Sema &S, SourceLocation Loc,





More information about the cfe-commits mailing list