[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