r342098 - Remove dead code made unnecessary by r342018.

Richard Smith via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 12 16:37:58 PDT 2018


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;




More information about the cfe-commits mailing list