[PATCH] D21057: [yaml] Add a ScalarTraits for mapping endian aware types.

Zachary Turner via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 7 08:46:19 PDT 2016


The difference is that without this patch, the following does not compile:

ulittle32_t X;
IO.mapRequired("u32", X);

Sure, I could map it to a uint32_t, but all my internal data structures are
represented with iendian aware types, so this allows me to interact with
existing code more cleanly
On Tue, Jun 7, 2016 at 8:04 AM David Majnemer <david.majnemer at gmail.com>
wrote:

> majnemer added a comment.
>
> I'm a little confused.  It seems to me that this wraps the endian-aware
> types so perfectly that there is not an effective difference between
> ulittle32_t and uint32_t.
>
>
> http://reviews.llvm.org/D21057
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160607/ed9a2389/attachment.html>


More information about the llvm-commits mailing list