[cfe-commits] r103350 - /cfe/trunk/lib/Sema/SemaOverload.cpp

Douglas Gregor dgregor at apple.com
Sat May 8 13:20:05 PDT 2010


Author: dgregor
Date: Sat May  8 15:20:05 2010
New Revision: 103350

URL: http://llvm.org/viewvc/llvm-project?rev=103350&view=rev
Log:
Don't destroy the data associated with an overload resolution candidate; it's ASTContext-allocated now

Modified:
    cfe/trunk/lib/Sema/SemaOverload.cpp

Modified: cfe/trunk/lib/Sema/SemaOverload.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaOverload.cpp?rev=103350&r1=103349&r2=103350&view=diff
==============================================================================
--- cfe/trunk/lib/Sema/SemaOverload.cpp (original)
+++ cfe/trunk/lib/Sema/SemaOverload.cpp Sat May  8 15:20:05 2010
@@ -341,7 +341,7 @@
       
   case Sema::TDK_Inconsistent:
   case Sema::TDK_InconsistentQuals:
-    delete static_cast<DFIParamWithArguments*>(Data);
+    // FIXME: Destroy the data?
     Data = 0;
     break;
 
@@ -459,12 +459,6 @@
 }
 
 void OverloadCandidateSet::clear() {
-  // 
-  for (iterator C = begin(), CEnd = end(); C != CEnd; ++C) {
-    if (C->FailureKind == ovl_fail_bad_deduction)
-      C->DeductionFailure.Destroy();
-  }
-       
   inherited::clear();
   Functions.clear();
 }





More information about the cfe-commits mailing list