arphaman added a comment. In https://reviews.llvm.org/D37291#857727, @klimek wrote:
> Thanks! I think this makes the code easier to understand. Now my remaining > question is why the ResultType is templates vs. also being an interface > (sorry for being late, was out on vacation the past 2 weeks :) ResultType is typically a vector of `AtomicChange` or `SymbolOccurrences` which we can't subclass from an interface. Or are you wondering if `AtomicChange` and `SymbolOccurrence` should subclass from a common interface? It could potentially be useful, but I'm not sure if there could be that many methods that would go into the interface. https://reviews.llvm.org/D37291 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits