[PATCH] D114833: [modules] Fix ambiguous name lookup for enum constants from hidden submodules.

Volodymyr Sapsai via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 30 17:14:17 PST 2021


vsapsai added inline comments.


================
Comment at: clang/lib/Sema/SemaDecl.cpp:16216-16223
                 if (!getLangOpts().CPlusPlus) {
                   // Postpone making the old definition visible until after we
                   // complete parsing the new one and do the structural
                   // comparison.
                   SkipBody->CheckSameAsPrevious = true;
                   SkipBody->New = createTagFromNewDecl();
                   SkipBody->Previous = Def;
----------------
Here is the early return the added comment refers to.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D114833/new/

https://reviews.llvm.org/D114833



More information about the cfe-commits mailing list