[Patches][RFC] What to do about bitcode streaming.

JF Bastien jfb at chromium.org
Fri Dec 19 12:59:07 PST 2014


Hi Rafael,

Would you mind waiting for Derek to come back from vacation to discuss
this? We do use this code and could improve how it's used and tested within
LLVM. Derek is the best person to discuss this, he'll be back in
mid-January.

Thanks,

JF

On Fri, Dec 19, 2014 at 6:41 AM, Rafael EspĂ­ndola <
rafael.espindola at gmail.com> wrote:
>
> > I CC'ed llvmdev to put a few more eyes on the "what's the right
> > direction?" question.
> >
> > IMO these both look like huge improvements.  The streaming interface was
> > the strangest part of the bitcode reader when I was trying to figure out
> > how it all fit together for the use-list-order work.
> >
> > Personally I favour the "just delete it" path, but maybe there's
> > something I'm missing, and the other patch looks great too.
> >
> > I didn't read carefully yet (although I noticed two quirks in the first
> > patch that I've pointed out below) -- I'll have a closer look once
> > you've decided on a direction.
>
> Thanks. Looks like the clang-format I had installed was a bit too old.
> I have also rebased and removed the debug output.
>
> Cheers,
> Rafael
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20141219/31ad1c5c/attachment.html>


More information about the llvm-commits mailing list