Fix a comment typo and add a couple suggestions from Chris.

llvm-svn: 49066
This commit is contained in:
Steve Naroff 2008-04-02 00:39:51 +00:00
parent 1f17cce2eb
commit 9b94b17806
2 changed files with 4 additions and 3 deletions

View File

@ -99,7 +99,8 @@ class Sema : public Action {
/// with @compatibility_alias, so that we can emit errors on duplicates and
/// find the declarations when needed. This construct is ancient and will
/// likely never be seen. Nevertheless, it is here for compatibility.
typedef llvm::DenseMap<IdentifierInfo*, ObjCCompatibleAliasDecl*> ObjCAliasTy;
typedef llvm::DenseMap<const IdentifierInfo*,
ObjCCompatibleAliasDecl*> ObjCAliasTy;
ObjCAliasTy ObjCAliasDecls;
// Enum values used by KnownFunctionIDs (see below).

View File

@ -42,7 +42,7 @@ Sema::DeclTy *Sema::isTypeName(const IdentifierInfo &II, Scope *S) const {
if (getLangOptions().ObjC1) {
// @interface and @compatibility_alias result in new type references.
// Creating a class alias is *extremely* rare.
ObjCAliasTy::const_iterator I = ObjCAliasDecls.find((IdentifierInfo*)&II);
ObjCAliasTy::const_iterator I = ObjCAliasDecls.find(&II);
if (I != ObjCAliasDecls.end())
return I->second->getClassInterface();
}
@ -141,7 +141,7 @@ Decl *Sema::LookupDecl(IdentifierInfo *II, unsigned NSI,
if (getLangOptions().ObjC1) {
// @interface and @compatibility_alias introduce typedef-like names.
// Unlike typedef's, they can only be introduced at file-scope (and are
// not therefore not scoped decls). They can, however, be shadowed by
// therefore not scoped decls). They can, however, be shadowed by
// other names in IDNS_Ordinary.
ObjCAliasTy::iterator I = ObjCAliasDecls.find(II);
if (I != ObjCAliasDecls.end())