• svenpanne@chromium.org's avatar
    Re-land CL 9466047. · 5033b9b6
    svenpanne@chromium.org authored
    Main change from the original CL: Call::ComputeTarget does not use IsProperty
    anymore, because this would potentially need a holder, which we don't have
    here. Using Map::LookupInDescriptors with a NULL holder is a bit fishy in
    general, because one has to be *extremely* careful when using its LookupResult.
    
    The original CL made Chrome's NetInternalsTest.netInternalsTourTabs browser test
    fail, but it's a mystery how this could happen: We should never reach
    Call::ComputeTarget via Call::RecordTypeFeedback with a CALLBACKS property,
    because we never consider calls to them monomorphic, which is in turn because of
    the stub cache leaving them in the pre-monomorphic state. Therefore, I don't
    have a clue how to write a regression test for this...
    
    As an additional tiny bonus, the --trace-opt output for deoptimizations has been
    improved.
    
    Review URL: https://chromiumcodereview.appspot.com/9584003
    
    git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10906 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
    5033b9b6
Name
Last commit
Last update
benchmarks Loading commit data...
build Loading commit data...
include Loading commit data...
preparser Loading commit data...
samples Loading commit data...
src Loading commit data...
test Loading commit data...
tools Loading commit data...
.gitignore Loading commit data...
AUTHORS Loading commit data...
ChangeLog Loading commit data...
LICENSE Loading commit data...
LICENSE.strongtalk Loading commit data...
LICENSE.v8 Loading commit data...
LICENSE.valgrind Loading commit data...
Makefile Loading commit data...
SConstruct Loading commit data...