[PATCH] D114971: [clang][deps] Handle symlinks in minimizing FS

2022-01-21 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 rG8cc2a1372704: [clang][deps] Handle symlinks in minimizing FS (authored by jansvoboda11). Changed prior to commit: https://reviews.llvm.org/D114971

[PATCH] D114971: [clang][deps] Handle symlinks in minimizing FS

2021-12-20 Thread Duncan P. N. Exon Smith via Phabricator via cfe-commits
dexonsmith accepted this revision. dexonsmith added a comment. This revision is now accepted and ready to land. LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114971/new/ https://reviews.llvm.org/D114971 __

[PATCH] D114971: [clang][deps] Handle symlinks in minimizing FS

2021-12-20 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 updated this revision to Diff 395399. jansvoboda11 added a comment. Prevent eager minimization of not-to-be-minimized files. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114971/new/ https://reviews.llvm.org/D114971 Files: clang/inc

[PATCH] D114971: [clang][deps] Handle symlinks in minimizing FS

2021-12-17 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 updated this revision to Diff 395104. jansvoboda11 added a comment. Rebase on top of new version of D114966 . Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114971/new/ https://reviews.llvm.org/D114971

[PATCH] D114971: [clang][deps] Handle symlinks in minimizing FS

2021-12-06 Thread Alex Lorenz via Phabricator via cfe-commits
arphaman accepted this revision. arphaman added a comment. This revision is now accepted and ready to land. This looks good to me! I agree, an `enum class` would be cleaner though. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114971/new/ https://r

[PATCH] D114971: [clang][deps] Handle symlinks in minimizing FS

2021-12-02 Thread Thorsten via Phabricator via cfe-commits
tschuett added inline comments. Comment at: clang/include/clang/Tooling/DependencyScanning/DependencyScanningFilesystem.h:229 /// The information requested from \c getOrCreateFileSystemEntry. enum RequestedInformation { +RI_UniqueID, Do you want to mod

[PATCH] D114971: [clang][deps] Handle symlinks in minimizing FS

2021-12-02 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 updated this revision to Diff 391371. jansvoboda11 added a comment. Rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114971/new/ https://reviews.llvm.org/D114971 Files: clang/include/clang/Tooling/DependencyScanning/DependencySc

[PATCH] D114971: [clang][deps] Handle symlinks in minimizing FS

2021-12-02 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 created this revision. jansvoboda11 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Depends on D114968 . Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D114971 Files: clan