kbobyrev added a comment. In https://reviews.llvm.org/D52084#1238537, @sammccall wrote:
> This change seems fine but... > > This representation with the raw DocIDs and position dumped seems mostly > useful for debugging buggy iterator implementations, but not really useful > for understanding query structure and behavior. I agree; There is another issue that I was looking into: I think that it might be useful to understand the structure of fuzzy find query when using `dexp` tool and I thought that it would be great if we could dump the iterator tree structure along with the results (which is an extension of https://reviews.llvm.org/D52233). For `dexp` usecase, it would be great to dump the size and the origin of each piece of iterator tree (e.g. labels), but I also think that it might be useful to have "debugging" format so I couldn't figure out what's the best approach here. > I thought we might be replacing this soon. Of course there's no fundamental > reason we can't keep both but I'm curious why this is an area of focus :-) Yes, we are. Initially, this wasn't an area of focus: I just forgot to update the comment in `Iterator.h` when moving `PostingList` to a separate file and slightly changing the format, but then Eric had a good proposal and I thought that it's a good improvement. Also, even though the implementation will be different, the dumping format could be the same, so it's not really implementation-specific. https://reviews.llvm.org/D52084 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
