Commit a52ffd44 authored by ishell's avatar ishell Committed by Commit bot

[ic] Experiment: disable map-specific handler cache.

IC data handlers support most of the hot cases nowdays. Let's see if
the map-specific code cache still help us to improve things.

BUG=v8:5561

Review-Url: https://codereview.chromium.org/2462973003
Cr-Commit-Position: refs/heads/master@{#40685}
parent c1753f5c
......@@ -18,6 +18,9 @@ Handle<Code> PropertyHandlerCompiler::Find(Handle<Name> name,
Handle<Map> stub_holder,
Code::Kind kind,
CacheHolderFlag cache_holder) {
// TODO(ishell): Experiment: don't cache handlers in map-specific code cache.
return Handle<Code>();
Code::Flags flags = Code::ComputeHandlerFlags(kind, cache_holder);
Code* code = stub_holder->LookupInCodeCache(*name, flags);
if (code == nullptr) return Handle<Code>();
......
......@@ -9992,6 +9992,8 @@ class CodeCache : public AllStatic {
void Map::UpdateCodeCache(Handle<Map> map,
Handle<Name> name,
Handle<Code> code) {
// TODO(ishell): Experiment: don't cache handlers in map-specific code cache.
return;
Isolate* isolate = map->GetIsolate();
Handle<FixedArray> cache(map->code_cache(), isolate);
Handle<FixedArray> new_cache = CodeCache::Put(isolate, cache, name, code);
......
......@@ -40,6 +40,9 @@ static Handle<Code> GetDummyCode(Isolate* isolate) {
} // namespace
TEST(CodeCache) {
// TODO(ishell): Experiment: don't cache handlers in map-specific code cache.
return;
CcTest::InitializeVM();
Isolate* isolate = CcTest::i_isolate();
Factory* factory = isolate->factory();
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment