[llvm-commits] [REVIEW] Fix an Aliasing Bug

Chandler Carruth chandlerc at google.com
Wed Jan 9 18:12:05 PST 2013


Using a union for type punning is a C feature, not a C++ feature.

Please use memcpy to move the underlying storage between well typed
variables. That is the best C++ pattern for this.


On Wed, Jan 9, 2013 at 8:33 AM, <dag at cray.com> wrote:

> Anton Korobeynikov <anton at korobeynikov.info> writes:
>
> >>> It looks like you may have forgotten the patch.
> >>
> >> This is the cover-letter from git send-email.  The actual patch is sent
> >> as a reply.
> > Where? There is no reply. Will you please provide a link to the email
> > in question?
>
> Found it:
>
>
> http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20130107/161398.html
>
> It doesn't show up in the threaded view in the archives.  None of my
> messages to llvm-commits shows up in the threaded view but they do
> appear in the author view in the archives.
>
> Curious.
>
>                              -David
> _______________________________________________
> 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/20130109/091982c5/attachment.html>


More information about the llvm-commits mailing list