[llvm-commits] [PATCH] Fix Alias Bug

[dag at cray.com]

David Blaikie <dblaikie at gmail.com> writes:

> General feedback: please send all patches for review as attachments,
> not inline, so that mail clients don't mangle your patches.

Ok, will fix the git config.

> It's still UB to access the non-active member of a union. I believe
> the only valid way to do this sort of thing is with memcpy (you're
> allowed to read/write the bytes of such simple types through char*s) &
> let the compiler optimize that into the same trivial copy.

Hmm.  Are you use this is UB?  It's the technique "everyone" seems to
recommend.  Unfortunately, I recently recycled my copy of the standard
in favor of the AVX2 spec.  :)

                               -David