This is an archive of the discontinued LLVM Phabricator instance.

[Tooling/Inclusion] Move the generated mapping .inc file to private places.
ClosedPublic

Authored by hokein on Feb 6 2023, 7:11 AM.

Details

Summary

The .inc files are private now, clients should tooling::stdlib APIs instead.

Diff Detail

Event Timeline

hokein created this revision.Feb 6 2023, 7:11 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 6 2023, 7:11 AM
hokein requested review of this revision.Feb 6 2023, 7:11 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 6 2023, 7:11 AM
kadircet accepted this revision.Feb 6 2023, 9:17 AM
This revision is now accepted and ready to land.Feb 6 2023, 9:17 AM
This revision was landed with ongoing or failed builds.Feb 7 2023, 1:07 AM
This revision was automatically updated to reflect the committed changes.
hokein added a comment.Feb 7 2023, 2:21 PM

Hi @hokein, this broke the LLDB incremental build (https://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake/50629/console). How do we fix it?

sorry for the breakage. Could you try to remove the two textual header "Tooling/Inclusions/{C,Std}SymbolMap.inc"lines in the clang/include/clang/module.modulemap to see whether it fixes the issue?

hokein added a comment.Feb 7 2023, 2:24 PM

Hi @hokein, this broke the LLDB incremental build (https://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake/50629/console). How do we fix it?

sorry for the breakage. Could you try to remove the two textual header "Tooling/Inclusions/{C,Std}SymbolMap.inc"lines in the clang/include/clang/module.modulemap to see whether it fixes the issue?

OK, it was fixed in https://github.com/llvm/llvm-project/commit/4cbf4bb4dd17b813179467396c692180681cb003.