Index: lib/Lex/PPDirectives.cpp =================================================================== --- lib/Lex/PPDirectives.cpp +++ lib/Lex/PPDirectives.cpp @@ -746,7 +746,8 @@ ModuleMap::KnownHeader *SuggestedModule, bool SkipCache) { Module *RequestingModule = getModuleForLocation(FilenameLoc); - bool RequestingModuleIsModuleInterface = !SourceMgr.isInMainFile(FilenameLoc); + bool RequestingModuleIsModuleInterface = + !SourceMgr.isInMainFile(FilenameLoc) && getLangOpts().CompilingModule; // If the header lookup mechanism may be relative to the current inclusion // stack, record the parent #includes. Index: test/VFS/Inputs/Nonmodular/A.h =================================================================== --- test/VFS/Inputs/Nonmodular/A.h +++ test/VFS/Inputs/Nonmodular/A.h @@ -0,0 +1 @@ +// A.h Index: test/VFS/Inputs/Nonmodular/Nonmodular.modulemap =================================================================== --- test/VFS/Inputs/Nonmodular/Nonmodular.modulemap +++ test/VFS/Inputs/Nonmodular/Nonmodular.modulemap @@ -0,0 +1,5 @@ +framework module Nonmodular [extern_c] { + umbrella header "umbrella.h" + export * + module * { export * } +} Index: test/VFS/Inputs/Nonmodular/nonmodular-headers.yaml =================================================================== --- test/VFS/Inputs/Nonmodular/nonmodular-headers.yaml +++ test/VFS/Inputs/Nonmodular/nonmodular-headers.yaml @@ -0,0 +1,34 @@ +{ + 'version': 0, + 'case-sensitive': 'false', + 'ignore-non-existent-contents': 'true', + 'roots': [ + { + 'type': 'directory', + 'name': "VDIR/Nonmodular.framework/Headers", + 'contents': [ + { + 'type': 'file', + 'name': "umbrella.h", + 'external-contents': "IN_DIR/Inputs/Nonmodular/umbrella.h" + }, + { + 'type': 'file', + 'name': "A.h", + 'external-contents': "IN_DIR/Inputs/Nonmodular/A.h" + } + ] + }, + { + 'type': 'directory', + 'name': "VDIR/Nonmodular.framework/Modules", + 'contents': [ + { + 'type': 'file', + 'name': "module.modulemap", + 'external-contents': "OUT_DIR/module.modulemap" + } + ] + } + ] +} Index: test/VFS/Inputs/Nonmodular/test.c =================================================================== --- test/VFS/Inputs/Nonmodular/test.c +++ test/VFS/Inputs/Nonmodular/test.c @@ -0,0 +1,3 @@ +// expected-no-diagnostics + +#include "umbrella.h" Index: test/VFS/Inputs/Nonmodular/umbrella.h =================================================================== --- test/VFS/Inputs/Nonmodular/umbrella.h +++ test/VFS/Inputs/Nonmodular/umbrella.h @@ -0,0 +1,5 @@ +#ifndef __umbrella_h__ +#define __umbrella_h__ + +#include +#endif Index: test/VFS/test_nonmodular.c =================================================================== --- test/VFS/test_nonmodular.c +++ test/VFS/test_nonmodular.c @@ -0,0 +1,11 @@ +// REQUIRES: shell + +// RUN: rm -rf %t +// RUN: mkdir -p %t/vdir %t/cache %t/outdir +// We can't have module.map inside Inputs/Nonmodular. +// RUN: cp %S/Inputs/Nonmodular/Nonmodular.modulemap %t/outdir/module.modulemap +// +// RUN: sed -e "s:VDIR:%t/vdir:g" -e "s:IN_DIR:%S:g" -e "s:OUT_DIR:%t/outdir:g" %S/Inputs/Nonmodular/nonmodular-headers.yaml > %t/vdir/nonmodular-headers.yaml +// RUN: %clang_cc1 -fmodule-name=Nonmodular -fmodules -Wnon-modular-include-in-framework-module -verify -fimplicit-module-maps -fmodules-cache-path=%t/cache -ivfsoverlay %t/vdir/nonmodular-headers.yaml -I %S/Inputs -F %t/vdir -fsyntax-only %S/Inputs/Nonmodular/test.c + +// expected-no-diagnostics