[PATCH] D142501: [clang][deps] Fix modulemap file path for implementation of module

Ben Langmuir via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 24 12:54:16 PST 2023


benlangmuir created this revision.
benlangmuir added a reviewer: jansvoboda11.
Herald added a project: All.
benlangmuir requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.

Use the name "as requested" for the path of the implemented module's modulemap file, just as we do for other modulemap file paths. This fixes fatal errors with modules where we tried to find framework headers relative to the wrong directory when imported by an implementation file of the same module.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D142501

Files:
  clang/lib/Tooling/DependencyScanning/ModuleDepCollector.cpp
  clang/test/ClangScanDeps/modules-implementation-vfs.m

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D142501.491880.patch
Type: text/x-patch
Size: 6235 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20230124/7030f6db/attachment.bin>


More information about the cfe-commits mailing list