malaperle added a comment. In https://reviews.llvm.org/D47272#1109914, @ilya-biryukov wrote:
> In https://reviews.llvm.org/D47272#1109909, @malaperle wrote: > > > > We do not to rely on symbols from the main file anyway, since any info > > > hat those provide can always be taken from the AST. > > > > I'll be adding those soon for workspace symbols... And also for document > > symbols. > > > I can add extra code to build pieces for the AST later. This is not hard to > do, but would require rearranging some code a bit more. > Will try to send the change tomorrow for review tomorrow. Does that SG? Sounds good. Doesn't have to be tomorrow :) Just making sure we're not on incompatible paths. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D47272 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
