[clang] [Sema]Use tag name lookup for class names (PR #112166)

Erich Keane via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 14 09:09:24 PDT 2024


=?utf-8?q?Gábor?= Spaits <gaborspaits1 at gmail.com>,Gabor Spaits
 <gaborspaits1 at gmail.com>,
=?utf-8?q?Gábor?= Spaits <gaborspaits1 at gmail.com>,
=?utf-8?q?Gábor?= Spaits <gaborspaits1 at gmail.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/112166 at github.com>


================
@@ -357,10 +357,13 @@ ParsedType Sema::getTypeName(const IdentifierInfo &II, SourceLocation NameLoc,
       return nullptr;
   }
 
-  // FIXME: LookupNestedNameSpecifierName isn't the right kind of
-  // lookup for class-names.
-  LookupNameKind Kind = isClassName ? LookupNestedNameSpecifierName :
-                                      LookupOrdinaryName;
+  // In case we know that the identifier is a class name, we know that it is
----------------
erichkeane wrote:

Absolutely not a problem, happens all the time.  Thank you very much for the patch!  Let us know if you need someone to commit this for you, or if you already have merge rights.

https://github.com/llvm/llvm-project/pull/112166


More information about the cfe-commits mailing list