After looking more, I think that trying to patch ia32 sNaN's into the heap
exposes fragile behavior, including compiler version dependencies. Seems much
cleaner to do this on the end target after de-ser.

New CL: https://chromiumcodereview.appspot.com/10093007/, properly tested this
time.

If you feel strongly otherwise I have another (un-published) CL here:
https://chromiumcodereview.appspot.com/10093008, but I am pretty uncomfortable
with this one, as it's a workaround for a bad fix, IMO. But it does work.



https://chromiumcodereview.appspot.com/9910029/

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to