[PATCH] Refactor DynTypedMatcher into a value type class, just like Matcher<T>.

Manuel Klimek klimek at google.com
Sat Oct 5 01:46:15 PDT 2013

Comment at: include/clang/ASTMatchers/ASTMatchersInternal.h:526
@@ +525,3 @@
+  StorageType Storage;
I have to cringe when I look at that implementation. I most definitely would hope to never have to touch it.

It's very complex and hard to understand, and I don't see the big benefits apart from having a workaround for a VSC++ bug.


More information about the cfe-commits mailing list