[PATCH] [C++11] Change DebugInfoFinder to use range-based loops

David Blaikie dblaikie at gmail.com
Sun Mar 16 07:43:01 PDT 2014


Looks fine to me - please commit

On Sun, Mar 16, 2014 at 2:20 AM, Alon Mishne <alonmishne at gmail.com> wrote:
> This patch changes DebugInfoFinder iterator functions to be
> range-based-loops-friendly. To make it even easier to users the iterators
> now also actually return the DI type, not just the MDNode - all but one user
> (in DebugIR) benefit from this.
>
> Notice that this means there are now 5 iterator typedefs instead of just
> one; but in the rare case they need to be referred to, such as in
> Cloning.cpp, "auto" can just be used. Also, once all supported compilers
> implement alias template, all 5 typedefs could be generalized into one.
>
> - Alon



More information about the llvm-commits mailing list