[Lldb-commits] [PATCH] D148776: [Modules] Move modulemaps to header search directories. NFC intended.
Volodymyr Sapsai via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Wed Apr 19 20:05:37 PDT 2023
vsapsai created this revision.
vsapsai added reviewers: EricWF, aprantl, chapuni.
Herald added subscribers: ributzka, s.egerton, simoncook, asb, fedor.sergeev, dschuff.
Herald added a reviewer: deadalnix.
Herald added a project: All.
vsapsai requested review of this revision.
Herald added subscribers: llvm-commits, lldb-commits, cfe-commits, jplehr, pcwang-thead, sstefan1, aheejin.
Herald added a reviewer: jdoerfert.
Herald added projects: clang, LLDB, LLVM.
In code we use `#include "llvm/Lib/Header.h"` which is located in
"llvm/include/llvm/Lib/Header.h", so we use "llvm/include/" as a header
search path. We should put modulemaps in the same directory and
shouldn't rely on clang to search in immediate subdirectories.
rdar://106677321
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D148776
Files:
clang/include/clang-c/module.modulemap
clang/include/clang/module.modulemap
clang/include/module.modulemap
clang/lib/Lex/HeaderSearch.cpp
lldb/include/lldb/module.modulemap
lldb/include/module.modulemap
llvm/CMakeLists.txt
llvm/include/CMakeLists.txt
llvm/include/llvm-c/module.modulemap
llvm/include/llvm/CMakeLists.txt
llvm/include/llvm/module.extern.modulemap
llvm/include/llvm/module.install.modulemap
llvm/include/llvm/module.modulemap
llvm/include/llvm/module.modulemap.build
llvm/include/module.extern.modulemap
llvm/include/module.install.modulemap
llvm/include/module.modulemap
llvm/include/module.modulemap.build
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D148776.515192.patch
Type: text/x-patch
Size: 61511 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20230420/1147967b/attachment-0001.bin>
More information about the lldb-commits
mailing list