Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1352)

Unified Diff: test/cctest/test-api-fast-accessor-builder.cc

Issue 1474543004: Implement Fast Accessor Builder (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Review feedback; all of the easy stuff. Created 5 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: test/cctest/test-api-fast-accessor-builder.cc
diff --git a/test/cctest/test-api-fast-accessor-builder.cc b/test/cctest/test-api-fast-accessor-builder.cc
new file mode 100644
index 0000000000000000000000000000000000000000..81b5d0e0a60eb5a0d1f034e7497884fbcded3ee3
--- /dev/null
+++ b/test/cctest/test-api-fast-accessor-builder.cc
@@ -0,0 +1,112 @@
+// Copyright 2015 the V8 project authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+// TODO(jochen): Remove this after the setting is turned on globally.
+#define V8_IMMINENT_DEPRECATION_WARNINGS
+
+#include <stdlib.h>
+
+#include "include/v8.h"
+
+#include "src/api.h"
+#include "test/cctest/cctest.h"
+
+namespace {
+
+// These tests mean to exercise v8::FastAccessorBuilder. Since initially the
+// "native" accessor will get called, we need to 'warmup' any accessor first,
+// to make sure we're actually testing the v8::FastAccessorBuilder result.
+// To accomplish this, we will
+// - call each accesssor N times before the actual test.
+// - wrap that call in a function, so that all such calls will go
+// through a single call site.
+// - register a native accessor which is different from the build one
+// (so that our tests will always fail if we don't end up in the 'fast'
+// accessor)
+#define WARMUP(src) "for(i = 0; i < 2; i++) { " src " } "
+
+static void NativePropertyAccessor(
+ const v8::FunctionCallbackInfo<v8::Value>& info) {
+ info.GetReturnValue().Set(v8_num(123));
+}
+
+} // anonymous namespace
+
+
+// Build a simple "fast accessor" and verify that it is being called.
+TEST(FastAccessor) {
+ LocalContext env;
+ v8::Isolate* isolate = env->GetIsolate();
+ v8::HandleScope scope(isolate);
+
+ v8::Local<v8::FunctionTemplate> foo = v8::FunctionTemplate::New(isolate);
+
+ // Native accessor, bar, returns 123.
+ foo->PrototypeTemplate()->SetAccessorProperty(
+ v8_str("bar"),
+ v8::FunctionTemplate::New(isolate, NativePropertyAccessor));
+
+ // Fast accessor, barf, returns 124.
+ auto fab = v8::experimental::FastAccessorBuilder::New(isolate);
+ fab->ReturnValue(fab->IntegerConstant(124));
+ foo->PrototypeTemplate()->SetAccessorProperty(
+ v8_str("barf"), v8::FunctionTemplate::NewWithFastHandler(
+ isolate, NativePropertyAccessor, fab));
+
+ // Install foo on the global object.
+ CHECK(env->Global()
+ ->Set(env.local(), v8_str("foo"),
+ foo->GetFunction(env.local()).ToLocalChecked())
+ .FromJust());
+
+ // Wrap f.barf + IC warmup.
+ CompileRun(
+ "function barf() { f = new foo(); return f.barf }; " WARMUP("barf()"));
+
+ ExpectInt32("f = new foo(); f.bar", 123);
+ ExpectInt32("f = new foo(); f.barf", 123); // First call in this call site.
+ ExpectInt32("barf()", 124); // Call via warmed-up callsite.
+}
+
+
+void AddInternalFieldAccessor(v8::Isolate* isolate,
+ v8::Local<v8::Template> templ, const char* name,
+ int field_no) {
+ auto builder = v8::experimental::FastAccessorBuilder::New(isolate);
+ builder->ReturnValue(
+ builder->GetInternalField(builder->GetCallTarget(), field_no));
+ templ->SetAccessorProperty(v8_str(name),
+ v8::FunctionTemplate::NewWithFastHandler(
+ isolate, NativePropertyAccessor, builder));
+}
+
+
+// "Fast" accessor that accesses an internal field.
+TEST(FastAccessorWithInternalField) {
+ LocalContext env;
+ v8::Isolate* isolate = env->GetIsolate();
+ v8::HandleScope scope(isolate);
+
+ v8::Local<v8::ObjectTemplate> foo = v8::ObjectTemplate::New(isolate);
+ foo->SetInternalFieldCount(3);
+ AddInternalFieldAccessor(isolate, foo, "field0", 0);
+ AddInternalFieldAccessor(isolate, foo, "field1", 1);
+ AddInternalFieldAccessor(isolate, foo, "field2", 2);
+
+ // Create an instance w/ 3 internal fields, put in a string, a Smi, nothing.
+ v8::Local<v8::Object> obj = foo->NewInstance(env.local()).ToLocalChecked();
+ obj->SetInternalField(0, v8_str("Hi there!"));
+ obj->SetInternalField(1, v8::Integer::New(isolate, 4321));
+ CHECK(env->Global()->Set(env.local(), v8_str("obj"), obj).FromJust());
+
+ // Warmup.
+ CompileRun("function field0() { return obj.field0 }; " WARMUP("field0()"));
+ CompileRun("function field1() { return obj.field1 }; " WARMUP("field1()"));
+ CompileRun("function field2() { return obj.field2 }; " WARMUP("field2()"));
+
+ // Access fields.
+ ExpectString("field0()", "Hi there!");
+ ExpectInt32("field1()", 4321);
+ ExpectUndefined("field2()");
+}

Powered by Google App Engine
This is Rietveld 408576698