[PATCH] D47032: Support: Simplify endian stream interface. NFCI.

Zachary Turner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 18 13:20:12 PDT 2018


zturner added subscribers: pcc, zturner.
zturner added a comment.

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).

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


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D47032





More information about the llvm-commits mailing list