akyrtzi added a comment. In D65846#1619645 <https://reviews.llvm.org/D65846#1619645>, @jfb wrote:
> My current guess is that this part of the test: > > c-index-test -write-pch %t.h.pch %s -fmodules -fmodules-cache-path=%t.mcp > -Xclang -triple -Xclang x86_64-apple-darwin > > > Is expected to generate the `unknown type name` error, but when than happens > it ignores `-fmodules-cache-path=%t.mcp` and doesn't create the directory. > The next line expects the directory to exist, which is why it can't create > the lock file (because the directory it's trying to create it in doesn't > exist). `clang -fmodules -fmodules-cache-path=...` is supposed to create the directory for the cache path, including the parent directories, AFAIK. If this doesn't happen it is a behavior change (and undesirable IMO). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65846/new/ https://reviews.llvm.org/D65846 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
