<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Tue, Aug 29, 2017 at 10:55 AM Ilya Biryukov <<a href="mailto:ibiryukov@google.com">ibiryukov@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi,<div><br></div><div>Thanks for the design.</div><div><br></div><div>1. You briefly mention an in-memory mapping on top the index to support efficient lookup of index records.</div><div>Is it part of the available implementation? Do you think it would make sense to reuse it in other tools?</div><div><br></div><div>I'm asking because that seems like something most editors and IDEs are also interested in. Proper reuse of this implementation could prove useful.</div><div><br></div><div>2.  In clangd, we're not controlling the build step, instead building ASTs in-memory. We would rather store the indexing information in-memory or consume it on the go while building ASTs.</div><div>Do you have suggestions on which parts of the API we should look at?</div><div>We could implement our own IndexASTConsumer, but are there more opportunities for reusing other parts of your implementation? Code for collecting indexing dependencies, definitions of high-level record structures (i.e. symbol definitions, etc.)?</div></div></blockquote><div><br></div><div>Btw, I'd assume in clangd we'll want both: indexing as part of the build, and updated (possibly overlayed) indexing as part of the AST reparsing.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><br></div></div><div class="gmail_extra"></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 29, 2017 at 9:34 AM, Manuel Klimek <span dir="ltr"><<a href="mailto:klimek@google.com" target="_blank">klimek@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi, thanks for the details design, I'm really looking forward for having this :)<div><br></div><div>On a high level, I think this all makes lots of sense. </div><div>I have one question about how things get deleted: will record files for headers be deleted at some point, or will they be kept around, and you have to look at all unit files to see whether a given record file is still valid?</div><div><br></div><div>Cheers,</div><div>/Manuel<br><div><br></div></div><div><div class="m_-8568356562873895740h5"><br><div class="gmail_quote"><div dir="ltr">On Tue, Aug 29, 2017 at 2:18 AM Nathan Hawes <<a href="mailto:nhawes@apple.com" target="_blank">nhawes@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word;line-break:after-white-space">Hey everyone,<div><br></div><div>Xcode 9 shipped with index-while-building functionality based on enhancements to Clang that we’d like to upstream. Key among them is a new option, <font face="Menlo">-index-store-path</font>, that in addition to Clang's usual outputs, causes it to write out indexing data at the supplied path with minimal overhead.</div><div><br></div><div>While the current implementation is available at <a href="https://github.com/apple/swift-clang" target="_blank">https://github.com/apple/swift-clang</a>, we’d like to start by getting feedback on the high-level design, which you can read about here: <a href="https://docs.google.com/document/d/1cH2sTpgSnJZCkZtJl1aY-rzy4uGPcrI-6RrUpdATO2Q/edit?usp=sharing" target="_blank">https://docs.google.com/document/d/1cH2sTpgSnJZCkZtJl1aY-rzy4uGPcrI-6RrUpdATO2Q/edit?usp=sharing</a></div><div><br></div><div>Please let us know of any concerns, comments or questions you have regarding this feature.</div><div><br></div><div>Thanks!</div></div><div style="word-wrap:break-word;line-break:after-white-space"><div>Nathan</div></div></blockquote></div></div></div></div>
</blockquote></div><br><br clear="all"><div><br></div></div><div class="gmail_extra">-- <br><div class="m_-8568356562873895740gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>Regards,</div><div>Ilya Biryukov</div></div></div></div></div>
</div></blockquote></div></div>