[clang] [clang][ASTImporter] Not using primary context in lookup table (PR #118466)

Michael Buch via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 18 04:29:19 PST 2024


================
@@ -6082,6 +6084,7 @@ TEST_P(ASTImporterLookupTableTest, LookupSearchesInTheWholeRedeclChain) {
   auto Res = LT.lookup(N1, Name);
   ASSERT_EQ(Res.size(), 1u);
   EXPECT_EQ(*Res.begin(), A);
+  EXPECT_TRUE(LT.lookup(N2, Name).empty());
----------------
Michael137 wrote:

So prior to your change this lookup would've found `A`? Because the `LookupTable` would unconditionally use the primary context?

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


More information about the cfe-commits mailing list