[llvm] r238369 - AsmPrinter: Stop exposing underlying DIEValue list, NFC
Duncan P. N. Exon Smith
dexonsmith at apple.com
Thu May 28 12:02:13 PDT 2015
Thanks both. Fixed in r238456.
> On 2015-May-27, at 19:21, Rafael Espíndola <rafael.espindola at gmail.com> wrote:
> According to
> The preferred one is foo_begin/foo_end.
> On 27 May 2015 at 20:09, Duncan P. N. Exon Smith <dexonsmith at apple.com> wrote:
>>> On 2015 May 27, at 16:19, David Blaikie <dblaikie at gmail.com> wrote:
>>> (alternatively - (optionally add range-based APIs for anything you currently need begin/end for, and) skip begin/end in favor of using the range accessor below)
>> Yeah, it'd be nice to get rid of these entirely. But since I
>> tend to need to basically write them anyway to build the range
>> accessor I usually just expose them by default.
>> Probably easy to delete though. I'll have a go.
Not so easy yet, since we currently need to compute an index. I'll
have another go once the patch to convert `DIE::Value`s to a linked
list has landed.
More information about the llvm-commits