[clang] [clang][modules] Dependency Scanning: Temporarily Turn Off Negative Directory Caching (PR #134698)

Cyndy Ishida via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 7 13:10:43 PDT 2025


cyndyishida wrote:

Talked a bit offline, but the patch LGTM, assuming it is purely restoring the previous behavior before  https://github.com/llvm/llvm-project/commit/55323ca6c8b9d21d85591f3499b299b62463321f, which would result in performance parity with the release compiler. 

> I suggest we carry this patch downstream (and possibly only in the current release branch while working to fix the build system or projects in the next release).

I don't have a strong opinion on whether it's merged here or in the fork. But I do believe all changes should go to the development branch too, to avoid the human cost of investigating the same issue as if it's an independently new one. When there's a resolution to the underlying cause, reverts should go to both places as well.

https://github.com/llvm/llvm-project/pull/134698


More information about the cfe-commits mailing list