This is an archive of the discontinued LLVM Phabricator instance.

Differential D144894

[clang] Documents clang-scan-deps requirements.
ClosedPublic

Authored by Mordante on Feb 27 2023, 10:37 AM.

Details

Reviewers
ChuanqiXu
Commits
rG8dbeb41f8ebd: [clang] Documents clang-scan-deps requirements.
Summary

This was discussed in https://llvm.org/PR61006.

Diff Detail

Unit TestsFailed

TimeTest
140 mslibcxx CI Modules > llvm-libc++-shared-cfg-in.libcxx/ranges/range_adaptors/range_all::all.nodiscard.verify.cpp
160 mslibcxx CI Modules > llvm-libc++-shared-cfg-in.libcxx/ranges/range_adaptors/range_common_view::adaptor.nodiscard.verify.cpp
150 mslibcxx CI Modules > llvm-libc++-shared-cfg-in.libcxx/ranges/range_adaptors/range_reverse::adaptor.nodiscard.verify.cpp
300 mslibcxx CI Modules > llvm-libc++-shared-cfg-in.libcxx/ranges/range_adaptors/range_transform::adaptor.nodiscard.verify.cpp
2,470 mslibcxx CI Modules > llvm-libc++-shared-cfg-in.std/algorithms/alg_modifying_operations/alg_copy::ranges.copy.segmented.pass.cpp
View Full Test Results (22 Failed)

Event Timeline

Mordante created this revision.Feb 27 2023, 10:37 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 27 2023, 10:37 AM
Mordante requested review of this revision.Feb 27 2023, 10:37 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 27 2023, 10:37 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
Harbormaster completed remote builds in B216260: Diff 500843.Feb 27 2023, 11:14 AM
ChuanqiXu accepted this revision.Feb 27 2023, 5:58 PM

LGTM. Thanks!

This revision is now accepted and ready to land.Feb 27 2023, 5:58 PM
This revision was landed with ongoing or failed builds.Feb 28 2023, 9:16 AM
Closed by commit rG8dbeb41f8ebd: [clang] Documents clang-scan-deps requirements. (authored by Mordante). · Explain Why
This revision was automatically updated to reflect the committed changes.
Mordante added a commit: rG8dbeb41f8ebd: [clang] Documents clang-scan-deps requirements..

Revision Contents

PathSize
clang/
docs/
6 lines

Diff 500843

clang/docs/StandardCPlusPlusModules.rst

Show First 20 Lines • Show All 1,027 Lines • ▼ Show 20 Lines
And if the user want a finer-grained control for any reason, e.g., to scan the generated source files, And if the user want a finer-grained control for any reason, e.g., to scan the generated source files,
the user can choose to get the dependency information per file. For example: the user can choose to get the dependency information per file. For example:
.. code-block:: console .. code-block:: console
$ clang-scan-deps -format=p1689 -- <path-to-compiler-executable>/clang++ -std=c++20 impl_part.cppm -c -o impl_part.o $ clang-scan-deps -format=p1689 -- <path-to-compiler-executable>/clang++ -std=c++20 impl_part.cppm -c -o impl_part.o
.. warning::
The ``<path-to-compiler-executable>/clang++`` should point to the real
binary and not to a symlink. If it points to a symlink the include paths
will not be correctly resolved.
And we'll get: And we'll get:
.. code-block:: text .. code-block:: text
{ {
"revision": 0, "revision": 0,
"rules": [ "rules": [
{ {
▲ Show 20 Lines • Show All 142 Lines • Show Last 20 Lines