[PATCH] D58418: [clang][DirectoryWatcher] Upstream DirectoryWatcher

Fri Mar 15 00:24:13 PDT 2019

jkorous added a comment.

In D58418#1430160 <https://reviews.llvm.org/D58418#1430160>, @thakis wrote:

> Why is this needed for index-while-building? My mental model for index-while-building is that that streams out build index metadata as part of the regular compile. Why does that require watching directories?

You're right that this isn't necessary for the indexing phase. But we also provide API so clients can consume the index. This functionality is used for getting notifications about index data changes.

You can see it for example here:
https://github.com/apple/swift-clang/blob/stable/lib/IndexDataStore/IndexDataStore.cpp#L111

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58418/new/

https://reviews.llvm.org/D58418