<div dir="ltr">Hi Rafael,<div><br></div><div>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.</div><div><br></div><div>Thanks,</div><div><br></div><div>JF</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Dec 19, 2014 at 6:41 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">> I CC'ed llvmdev to put a few more eyes on the "what's the right<br>
> direction?" question.<br>
><br>
> IMO these both look like huge improvements.  The streaming interface was<br>
> the strangest part of the bitcode reader when I was trying to figure out<br>
> how it all fit together for the use-list-order work.<br>
><br>
> Personally I favour the "just delete it" path, but maybe there's<br>
> something I'm missing, and the other patch looks great too.<br>
><br>
> I didn't read carefully yet (although I noticed two quirks in the first<br>
> patch that I've pointed out below) -- I'll have a closer look once<br>
> you've decided on a direction.<br>
<br>
</span>Thanks. Looks like the clang-format I had installed was a bit too old.<br>
I have also rebased and removed the debug output.<br>
<br>
Cheers,<br>
Rafael<br>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div></div>