[PATCH] D45645: Remove faulty assertion in llvm-pdbutil

[Zachary Turner via llvm-commits]

If the tests pass, this is probably fine. Honestly this code is really
complex and while I think it’s useful, it may not be useful enough to
justify the complexity so I’m thinking about getting rid of it
On Fri, Apr 13, 2018 at 4:44 PM Adrian McCarthy via Phabricator <
reviews at reviews.llvm.org> wrote:

> amccarth added a comment.
>
> I meant to mention that the instance of the empty class is reported as 8
> bytes of padding (in a 64-bit build), in case that's relevant to anybody's
> understanding of the issue.
>
>
> https://reviews.llvm.org/D45645
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180414/a491b5ea/attachment.html>