[llvm-commits] [llvm] r172081 - /llvm/trunk/unittests/Support/YAMLIOTest.cpp

[dag at cray.com]

Jakub Staszak <kubastaszak at gmail.com> writes:

> Fixed in r172105.
>
> On Jan 10, 2013, at 8:23 PM, Dmitri Gribenko <gribozavr at gmail.com> wrote:
>
>> On Thu, Jan 10, 2013 at 8:17 PM, David Greene <greened at obbligato.org> wrote:
>>> Author: greened
>>> Date: Thu Jan 10 12:17:54 2013
>>> New Revision: 172081
>>> 
>>> URL: http://llvm.org/viewvc/llvm-project?rev=172081&view=rev
>>> Log:
>>> Fix Alias Bug
>>> 
>>> Use memcpy to do type punning instead of a cast.  A cast or similar
>>> operation through a union breaks strict aliasing rules.
>> 
>> Sorry, but it looks like it broke tests:
>> http://lab.llvm.org:8011/builders/clang-x86_64-debian-fast/builds/99

Thanks.  I did not get a failure e-mail for this.  Should I have
expected one?

                                 -David