[PATCH] [Bitcode reader] Fix a few assertions when reading invalid files

[Filipe Cabecinhas]

For now I'm still making sure we don't assert on stuff.
We can eventually add the Diag-style error handling, but that'll end up
“just” transforming these fixes I'm doing, it won't change them, so I'm
continuing to go through my set of inputs that crash the reader.

These ones also end up using the error reporting that was already there, so
it's not like they're report_fatal_error() calls. They actually tell the
caller that there was an error and it can do something (but not that much)
about it.

I might end up starting doing the Diag-style patch soon, but it depends on
the other work I have to do. In the meantime I'll keep going over the crash
fixes, since most of those are small-ish patches. (Unless there are
objections to me doing it now)

  Filipe

On Mon, Feb 2, 2015 at 3:12 PM, Sean Silva <chisophugis at gmail.com> wrote:

> are you ever planning on switching over to a more "diag" style interface
> like Rafael suggested? That would allow making these errors much more
> informative.
>
>
> http://reviews.llvm.org/D7325
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150202/8d33a541/attachment.html>