[PATCH] D38806: DepthFirstIterator.h: Use C++11 features to call a completed method onthe set type, instead of requiring that one exists.

[Daniel Berlin via Phabricator via llvm-commits]

dberlin added a comment.

I'll add a unit test.

FWIW, you can get rid of the overload hack, i just can't get it to work in all compilers.

You can now nice create a class that has a ::value of true or false if the method exists, but passing that as a template parameter doesn't work so well yet, and using constexpr functions to achieve the same goal doesn't quite work yet either.


https://reviews.llvm.org/D38806