[PATCH] D33245: Simplify the use of CVTypeVisitor

[Zachary Turner via Phabricator via llvm-commits]

zturner added a comment.

Note that there is one instance of using `CVTypeVisitor` left that I couldn't get rid of in this patch.  The one in `RandomAccessVisitorTest`.  I have another patch which depends on this one which makes some fairly significantly change in this area, and after that patch we can get rid of that.

Once that happens, I plan to completely delete `CVTypeVisitor` (or at least hide it in the implementation file), and the only thing people will use is these newly introduced free functions.


https://reviews.llvm.org/D33245