On Thu, Mar 13, 2014 at 3:12 PM, Rodolph Perfetta < [email protected]> wrote:
> [...] On a more serious notes we have no objection to the current patch > knowing the benefits. > OK, my proposal would be: Let's land the current CL, if something breaks mysteriously on the simulator runs, one can easily check with strictaliasing=off if this is really related to our current discussion (very unlikely, given all the other "unsafe" casts around already). I'll have a look if I can massage Instruction to reduce the casting horror a bit, but as I had to learn yesterday the hard way, there are some adventurous assumptions/"techniques" around this class, like relying on the C++ specification edge case of "sizeof(MyFunnyEmptyClass) == 1". As a first step, I'll remove that assumption, but in a separate CL. -- -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev --- You received this message because you are subscribed to the Google Groups "v8-dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/d/optout.
