[cfe-dev] Type::isIncompleteType()...

[Steve Naroff]

Folks,

Do you think it makes sense for Type::isIncompleteType() to return  
true when the Tag definition is invalid?

For example...

   case Tagged:
      // A tagged type (struct/union/enum/class) is incomplete if the  
decl is a
      // forward declaration, but not a full definition (C99 6.2.5p22).
-    return !cast<TagType>(CanonicalType)->getDecl()->isDefinition();
+    TagDecl *TD = cast<TagType>(CanonicalType)->getDecl();
+    return !TD->isDefinition() || TD->isInvalidDecl();

Since this predicate is fairly low-level, I wanted to get some  
feedback...

snaroff