majnemer added inline comments.

================
Comment at: test/std/utilities/meta/meta.type.synop/endian.pass.cpp:39-42
+    union {
+        uint32_t i;
+        char c[4];
+    } u = {0x01020304};
----------------
This is undefined behavior as-is because you are reading from a union member 
other than the active union member.

I'd recommend a sequence like:
  uint32_t i = 0x01020304;
  char c[4];
  memcpy(c, &i, 4);


https://reviews.llvm.org/D35472



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to