[PATCH] D142167: [clang][modules] Disallow importing private framework in the implementation

2023-01-20 Thread Jan Svoboda via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGc3efd52770ca: [clang][modules] Disallow importing private framework in the implementation (authored by jansvoboda11). Repository: rG LLVM Github

[PATCH] D142167: [clang][modules] Disallow importing private framework in the implementation

2023-01-19 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 added inline comments. Comment at: clang/lib/Sema/SemaModule.cpp:544 // FIXME: Should we warn on a redundant import of the current module? - if (Mod->getTopLevelModuleName() == getLangOpts().CurrentModule && + if (Mod->isForBuilding(getLangOpts()) &&

[PATCH] D142167: [clang][modules] Disallow importing private framework in the implementation

2023-01-19 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 created this revision. jansvoboda11 added a reviewer: benlangmuir. Herald added subscribers: ChuanqiXu, ributzka. Herald added a project: All. jansvoboda11 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Whenever we are