Thank you for the feedback.
There was a problem with the difference between the int32 addition the int64
address calculation - the 32 bit argument to the addition operation is not
sign
extended when used in an 64 bit address calculation operation so the
calculation
failed when it was negative. The constant offset is sign extended. It
appears
that a simple type check can avoid this by only applying the optimization
when
the argument to the 32 bit addition operation is not negative. Some source
comments have been added to explain this.
Can you see any other issues?
It may well be cleaner to transform the int32 addition to an int64 addition
earlier and I'll give this a try too.
https://codereview.chromium.org/860283004/
--
--
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.
