[PATCH] Introducing initial UseAuto transform
Edwin Vane
edwin.vane at intel.com
Fri Feb 15 14:30:21 PST 2013
Work in progress on safer iterator detection.
Based on our conversations, I've redone the iterator detection. This patch is
highly unpolished and I'm posting it for feedback on the algorithm. I now detect
only standard iterator names of standard containers or more sugared types. This
work is done all by matchers now. Some notes on the matchers:
- I'm thinking about contributing hasDeclContext, hasSpecifier, namesType to
the ASTMatchers library.
- Also included would be the 'has member getDecl' type traits test.
- hasDecl2 is similar to hasDecl except it's more generic. Is it safe to replace
the existing one that works only on typedefs? If so, how should we distinguish
it from hasDeclaration which has similar behaviour but looks through sugar.
- The matchers hasStd*Name() is really just to avoid needing more overloads of
allOf(). They're slightly more efficient than the alternative too.
Hi klimek,
http://llvm-reviews.chandlerc.com/D392
CHANGE SINCE LAST DIFF
http://llvm-reviews.chandlerc.com/D392?vs=958&id=1007#toc
Files:
cpp11-migrate/UseAuto/UseAutoActions.cpp
cpp11-migrate/UseAuto/UseAutoMatchers.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D392.4.patch
Type: text/x-patch
Size: 12446 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130215/49c5af4a/attachment.bin>
More information about the cfe-commits
mailing list