[PATCH] D34873: Fix miscompiled 32bit binaries by mingw

Tim Northover via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 28 09:29:17 PDT 2017


t.p.northover added inline comments.


================
Comment at: lib/AST/ExprConstant.cpp:583
+    uint64_t& GetArrayInitIndex() {
+        return reinterpret_cast<uint64_t&>(ArrayInitIndex[0]);
+    }
----------------
rnk wrote:
> Surely this will fault on SPARC or ARM or other ISAs that care about alignment?
As well as being horribly undefined behaviour.


https://reviews.llvm.org/D34873





More information about the cfe-commits mailing list