JonasToth added a comment.

> Looks like pointless code duplication that is easily avoidable.

True, but I would prefer the refactoring to be private then (so the template 
stuff with the boilerplate) and a simple overloading interface dispatching to 
the templated stuff.
In the end I think the public interface is cleaner, easier to understand and 
not so easily misused (as the template would accept everything in principle, 
and SFINAE would be worse).


Repository:
  rCTE Clang Tools Extra

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59650/new/

https://reviews.llvm.org/D59650



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to