[PATCH] D32547: [CodeView] Isolate the parsing code for various types of CV debug info fragments

Zachary Turner via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 26 16:48:24 PDT 2017


I actually already have writing done locally, just want to land a few
patches before I upload it
On Wed, Apr 26, 2017 at 4:39 PM Reid Kleckner via Phabricator <
reviews at reviews.llvm.org> wrote:

> rnk accepted this revision.
> rnk added a comment.
>
> In https://reviews.llvm.org/D32547#738775, @amccarth wrote:
>
> > I'm satisfied, but you might want to wait to hear from at least one
> other reviewer.
>
>
> Looks good, at a really really high level. I'd say when you get to the
> writing portion, try to get more feedback from Rui and Bob, since it's
> useful to have a linker perspective for that.
>
>
> https://reviews.llvm.org/D32547
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170426/1b57770c/attachment.html>


More information about the llvm-commits mailing list