Reviewers: Jakob,
Message:
Hi Jakob,
Here is the CL we discussed.
1) It duplicates too much code. Thinking about fixes.
2) I had to remove an optimization that went from MONOMORPHIC->GENERIC state
without missing to the runtime, because the stats need to be updated on that
transition.
3) There is a "test" to verify functionality, but I won't check this in,
it's
just for you to see.
thx,
--Michael
Description:
CallIC must update type feedback info correctly.
CallIC, as the first of vector-based ICs didn't update the ic with type
count counter and the generic count counter correctly. This CL fixes
that.
BUG=
Please review this at https://codereview.chromium.org/445943002/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files (+92, -15 lines):
M src/ia32/code-stubs-ia32.cc
M src/ic.h
M src/ic.cc
A test/mjsunit/REMOVE_callic_behavior.js
--
--
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.
