[PATCH] D34880: [analyzer] Make StmtDataCollector part of the CloneDetection API

2017-07-02 Thread Raphael Isemann via Phabricator via cfe-commits
teemperor added a comment. Yeah, we should move it, but it should land somewhere in a header in `/AST/` so that for example the StmtProfiler could make use of this. I'm open for suggestions here :) https://reviews.llvm.org/D34880 ___ cfe-commits m

[PATCH] D34880: [analyzer] Make StmtDataCollector part of the CloneDetection API

2017-07-02 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. Looks good. To think: maybe we need a better name (and/or a place to live) for this API, now that it's not only related to clone detection, but is useful for more stuff? Comment