[cfe-commits] r76337 - in /cfe/trunk: lib/AST/Type.cpp test/SemaTemplate/temp_class_spec_neg.cpp

[Argiris Kirtzidis]

Author: akirtzidis
Date: Sat Jul 18 16:18:10 2009
New Revision: 76337

URL: http://llvm.org/viewvc/llvm-project?rev=76337&view=rev
Log:
Re-enable 'test/SemaTemplate/temp_class_spec_neg.cpp', after commenting out the cause of the crash.

Modified:
    cfe/trunk/lib/AST/Type.cpp
    cfe/trunk/test/SemaTemplate/temp_class_spec_neg.cpp

Modified: cfe/trunk/lib/AST/Type.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/AST/Type.cpp?rev=76337&r1=76336&r2=76337&view=diff

==============================================================================
--- cfe/trunk/lib/AST/Type.cpp (original)
+++ cfe/trunk/lib/AST/Type.cpp Sat Jul 18 16:18:10 2009
@@ -57,7 +57,9 @@
 }
 
 void DependentSizedArrayType::Destroy(ASTContext& C) {
-  SizeExpr->Destroy(C);
+  // FIXME: Resource contention like in ConstantArrayWithExprType ?
+  // May crash, depending on platform or a particular build.
+  // SizeExpr->Destroy(C);
   this->~DependentSizedArrayType();
   C.Deallocate(this);
 }

Modified: cfe/trunk/test/SemaTemplate/temp_class_spec_neg.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaTemplate/temp_class_spec_neg.cpp?rev=76337&r1=76336&r2=76337&view=diff

==============================================================================
--- cfe/trunk/test/SemaTemplate/temp_class_spec_neg.cpp (original)
+++ cfe/trunk/test/SemaTemplate/temp_class_spec_neg.cpp Sat Jul 18 16:18:10 2009
@@ -1,8 +1,4 @@
-// FIXME: Disabled because this always crashes on some platforms.
-// RUNX: clang-cc -fsyntax-only -verify %s
-// RUN: false
-// XFAIL
-
+// RUN: clang-cc -fsyntax-only -verify %s
 template<typename T> struct vector;
 
 // C++ [temp.class.spec]p6: