LGTM, thanks for fixing this!
For the record, it would be kinda nice to have one regression test for each
class of instructions affected by the fix. But on the other hand, I'm not
sure
how much value this regression test provides in the long run, beyond proving
right now that the problem was an actual bug. So I'm fine with landing as
is.
https://codereview.chromium.org/194883003/
--
--
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.