[PATCH] D20122: [codeview] Add type stream merging prototype

Zachary Turner via llvm-commits llvm-commits at lists.llvm.org
Thu May 12 17:15:08 PDT 2016


Can't you make them mutable? I agree it's a bit annoying to change all
those classes, but it will probably need to happen eventually anyway
On Thu, May 12, 2016 at 5:11 PM Reid Kleckner <rnk at google.com> wrote:

> rnk added a comment.
>
> Hm, the latest changes to TypeVisitor made this a lot, lot harder. With
> the old visitor, I duplicated a lot of the layout knowledge, but now I'm
> having a hard time remapping all the type indices because the current high
> level record types are immutable.
>
>
> http://reviews.llvm.org/D20122
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160513/34c9ef41/attachment.html>


More information about the llvm-commits mailing list