<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, May 18, 2018 at 1:19 PM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Note that the endianness values can be passed in to the llvm functions at runtime, there are non template versions of the write functions that accept endianness as arguments (that may be exactly what you used, btw, I didn’t read the patch in detail).<br></blockquote><div><br></div><div>Yes, there are functions in Endian.h that take a pointer and an endianness, in this patch I added functions that take a stream instead of a pointer.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">It’s a shame this object hierarchy was developed separately from the BinaryStream / StreamReader / StreamWriter interfaces, because it seems like there’s some overlap and it would serve this use case nicely <br></blockquote><div><br></div><div>At some point we might want to investigate using memory mapped I/O instead of raw_ostream to write out object files. At that point we can decide if we want to use BinaryStream or something more lightweight.</div><div><br></div><div>Peter</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_quote"><div><div class="m_-2563489848217515608h5"><div dir="ltr">On Fri, May 18, 2018 at 12:50 PM Peter Collingbourne via Phabricator via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br></div></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="m_-2563489848217515608h5">This revision was automatically updated to reflect the committed changes.<br>
Closed by commit rLLD332757: Support: Simplify endian stream interface. NFCI. (authored by pcc, committed by ).<br>
<br>
Changed prior to commit:<br>
  <a href="https://reviews.llvm.org/D47032?vs=147396&id=147573#toc" rel="noreferrer" target="_blank">https://reviews.llvm.org/D4703<wbr>2?vs=147396&id=147573#toc</a><br>
<br>
Repository:<br>
  rLLD LLVM Linker<br>
<br>
<a href="https://reviews.llvm.org/D47032" rel="noreferrer" target="_blank">https://reviews.llvm.org/D4703<wbr>2</a><br>
<br>
Files:<br>
  wasm/WriterUtils.cpp<br>
<br>
<br>
Index: wasm/WriterUtils.cpp<br>
==============================<wbr>==============================<wbr>=======<br>
--- wasm/WriterUtils.cpp<br>
+++ wasm/WriterUtils.cpp<br>
@@ -70,7 +70,7 @@<br>
<br>
 void wasm::writeU32(raw_ostream &OS, uint32_t Number, const Twine &Msg) {<br>
   debugWrite(OS.tell(), Msg + "[0x" + utohexstr(Number) + "]");<br>
-  support::endian::Writer<suppor<wbr>t::little>(OS).write(Number);<br>
+  support::endian::write(OS, Number, support::little);<br>
 }<br>
<br>
 void wasm::writeValueType(raw_ostre<wbr>am &OS, uint8_t Type, const Twine &Msg) {<br>
<br>
<br></div></div>
______________________________<wbr>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-commits</a><br>
</blockquote></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="m_-2563489848217515608gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">-- <div>Peter</div></div></div>
</div></div>