<div dir="ltr">And on a separate thread [0] Derek indicated he'd be fine with removing it. I'll leave this thread open until end of Monday to receive any other opinions, then proceed to remove it.<div><br></div><div>Peter<br><div><br></div><div>[0] <a href="http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20161024/400754.html">http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20161024/400754.html</a><br></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 28, 2016 at 10:08 AM, Peter Collingbourne <span dir="ltr"><<a href="mailto:peter@pcc.me.uk" target="_blank">peter@pcc.me.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi all,<div><br></div><div>BitstreamReader is the only in-tree client of the DataStreamer and MemoryObject interfaces. In practice when using user-facing LLVM tools, the bitcode will normally either be in memory or in a file, so the best way to access it is through memory, either directly or memory mapped.</div><div><br></div><div>As part of some refactorings I am making to BitstreamReader, I would like to simplify it by changing it to use a memory buffer directly and then removing the DataStreamer and MemoryObject interfaces.</div><div><br></div><div>My understanding that the only significant user of that interface as designed (for network streaming) is PNaCl, however it looks like removing this interface would not impose a significant burden on that project; they are already maintaining a couple of forks of BitstreamReader [1] so it would appear that the only change necessary on their side would be to also fork the interfaces.</div><div><br></div><div>If you are aware of any other significant users, please speak up!</div><div><br></div><div>Thanks,<span class="HOEnZb"><font color="#888888"><br>-- <br><div class="m_8140976728020832372gmail-m_-5372179860632505484gmail_signature"><div dir="ltr">-- <div>Peter</div><div><br></div><div>[1] <a href="https://chromium.googlesource.com/native_client/pnacl-llvm/+/master/include/llvm/Bitcode/NaCl/NaClBitstreamReader.h" target="_blank">https://chromium.googlesou<wbr>rce.com/native_client/pnacl-<wbr>llvm/+/master/include/llvm/<wbr>Bitcode/NaCl/NaClBitstreamRead<wbr>er.h</a> <a href="https://chromium.googlesource.com/native_client/pnacl-subzero/+/master/pnacl-llvm/include/llvm/Bitcode/NaCl/NaClBitstreamReader.h" target="_blank">https://chromium.<wbr>googlesource.com/native_<wbr>client/pnacl-subzero/+/master/<wbr>pnacl-llvm/include/llvm/<wbr>Bitcode/NaCl/<wbr>NaClBitstreamReader.h</a></div></div></div>
</font></span></div></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">-- <div>Peter</div></div></div>
</div>