Author: rsmith Date: Wed Sep 12 16:37:58 2018 New Revision: 342098 URL: http://llvm.org/viewvc/llvm-project?rev=342098&view=rev Log: Remove dead code made unnecessary by r342018.
Modified: cfe/trunk/lib/Sema/SemaLookup.cpp Modified: cfe/trunk/lib/Sema/SemaLookup.cpp URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaLookup.cpp?rev=342098&r1=342097&r2=342098&view=diff ============================================================================== --- cfe/trunk/lib/Sema/SemaLookup.cpp (original) +++ cfe/trunk/lib/Sema/SemaLookup.cpp Wed Sep 12 16:37:58 2018 @@ -1409,13 +1409,6 @@ bool Sema::hasVisibleMergedDefinition(Na } bool Sema::hasMergedDefinitionInCurrentModule(NamedDecl *Def) { - // FIXME: When not in local visibility mode, we can't tell the difference - // between a declaration being visible because we merged a local copy of - // the same declaration into it, and it being visible because its owning - // module is visible. - if (Def->getModuleOwnershipKind() == Decl::ModuleOwnershipKind::Visible && - getLangOpts().ModulesLocalVisibility) - return true; for (const Module *Merged : Context.getModulesWithMergedDefinition(Def)) if (isInCurrentModule(Merged, getLangOpts())) return true; _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits