[llvm] r234359 - Use support::endian. NFC.

Sean Silva chisophugis at gmail.com
Tue Apr 7 16:19:47 PDT 2015


Neat. Didn't know we had this utility.

On Tue, Apr 7, 2015 at 2:22 PM, Rafael Espindola <rafael.espindola at gmail.com
> wrote:

> Author: rafael
> Date: Tue Apr  7 16:22:05 2015
> New Revision: 234359
>
> URL: http://llvm.org/viewvc/llvm-project?rev=234359&view=rev
> Log:
> Use support::endian. NFC.
>
> Modified:
>     llvm/trunk/include/llvm/MC/MCObjectWriter.h
>
> Modified: llvm/trunk/include/llvm/MC/MCObjectWriter.h
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/MC/MCObjectWriter.h?rev=234359&r1=234358&r2=234359&view=diff
>
> ==============================================================================
> --- llvm/trunk/include/llvm/MC/MCObjectWriter.h (original)
> +++ llvm/trunk/include/llvm/MC/MCObjectWriter.h Tue Apr  7 16:22:05 2015
> @@ -13,6 +13,7 @@
>  #include "llvm/ADT/SmallVector.h"
>  #include "llvm/Support/Compiler.h"
>  #include "llvm/Support/DataTypes.h"
> +#include "llvm/Support/EndianStream.h"
>  #include "llvm/Support/raw_ostream.h"
>  #include <cassert>
>
> @@ -120,33 +121,27 @@ public:
>    }
>
>    void WriteLE16(uint16_t Value) {
> -    Write8(uint8_t(Value >> 0));
> -    Write8(uint8_t(Value >> 8));
> +    support::endian::Writer<support::little>(OS).write(Value);
>    }
>
>    void WriteLE32(uint32_t Value) {
> -    WriteLE16(uint16_t(Value >> 0));
> -    WriteLE16(uint16_t(Value >> 16));
> +    support::endian::Writer<support::little>(OS).write(Value);
>    }
>
>    void WriteLE64(uint64_t Value) {
> -    WriteLE32(uint32_t(Value >> 0));
> -    WriteLE32(uint32_t(Value >> 32));
> +    support::endian::Writer<support::little>(OS).write(Value);
>    }
>
>    void WriteBE16(uint16_t Value) {
> -    Write8(uint8_t(Value >> 8));
> -    Write8(uint8_t(Value >> 0));
> +    support::endian::Writer<support::big>(OS).write(Value);
>    }
>
>    void WriteBE32(uint32_t Value) {
> -    WriteBE16(uint16_t(Value >> 16));
> -    WriteBE16(uint16_t(Value >> 0));
> +    support::endian::Writer<support::big>(OS).write(Value);
>    }
>
>    void WriteBE64(uint64_t Value) {
> -    WriteBE32(uint32_t(Value >> 32));
> -    WriteBE32(uint32_t(Value >> 0));
> +    support::endian::Writer<support::big>(OS).write(Value);
>    }
>
>    void Write16(uint16_t Value) {
>
>
> _______________________________________________
> 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/20150407/866401be/attachment.html>


More information about the llvm-commits mailing list