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

Unified Diff: src/builtins/builtins.cc

Issue 2803903002: Revert of [snapshot] Move builtins generation into mksnapshot (Closed)
Patch Set: rebased Created 3 years, 8 months 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
« no previous file with comments | « src/builtins/builtins.h ('k') | src/builtins/builtins-call.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/builtins/builtins.cc
diff --git a/src/builtins/builtins.cc b/src/builtins/builtins.cc
index 3ac9a198cc7a2b9ba67a99a325bbb1b369323de4..e9528319bde577f7a811a2ec3b4188e46049a535 100644
--- a/src/builtins/builtins.cc
+++ b/src/builtins/builtins.cc
@@ -7,6 +7,10 @@
#include "src/assembler-inl.h"
#include "src/builtins/builtins-descriptors.h"
#include "src/callable.h"
+#include "src/code-events.h"
+#include "src/compiler/code-assembler.h"
+#include "src/ic/ic-state.h"
+#include "src/interface-descriptors.h"
#include "src/isolate.h"
#include "src/macro-assembler.h"
#include "src/objects-inl.h"
@@ -25,6 +29,173 @@ Builtins::Builtins() : initialized_(false) {
Builtins::~Builtins() {}
+namespace {
+void PostBuildProfileAndTracing(Isolate* isolate, Code* code,
+ const char* name) {
+ PROFILE(isolate, CodeCreateEvent(CodeEventListener::BUILTIN_TAG,
+ AbstractCode::cast(code), name));
+#ifdef ENABLE_DISASSEMBLER
+ if (FLAG_print_builtin_code) {
+ CodeTracer::Scope trace_scope(isolate->GetCodeTracer());
+ OFStream os(trace_scope.file());
+ os << "Builtin: " << name << "\n";
+ code->Disassemble(name, os);
+ os << "\n";
+ }
+#endif
+}
+
+typedef void (*MacroAssemblerGenerator)(MacroAssembler*);
+typedef void (*CodeAssemblerGenerator)(compiler::CodeAssemblerState*);
+
+Code* BuildWithMacroAssembler(Isolate* isolate,
+ MacroAssemblerGenerator generator,
+ Code::Flags flags, const char* s_name) {
+ HandleScope scope(isolate);
+ const size_t buffer_size = 32 * KB;
+ byte buffer[buffer_size]; // NOLINT(runtime/arrays)
+ MacroAssembler masm(isolate, buffer, buffer_size, CodeObjectRequired::kYes);
+ DCHECK(!masm.has_frame());
+ generator(&masm);
+ CodeDesc desc;
+ masm.GetCode(&desc);
+ Handle<Code> code =
+ isolate->factory()->NewCode(desc, flags, masm.CodeObject());
+ PostBuildProfileAndTracing(isolate, *code, s_name);
+ return *code;
+}
+
+Code* BuildAdaptor(Isolate* isolate, Address builtin_address,
+ Builtins::ExitFrameType exit_frame_type, Code::Flags flags,
+ const char* name) {
+ HandleScope scope(isolate);
+ const size_t buffer_size = 32 * KB;
+ byte buffer[buffer_size]; // NOLINT(runtime/arrays)
+ MacroAssembler masm(isolate, buffer, buffer_size, CodeObjectRequired::kYes);
+ DCHECK(!masm.has_frame());
+ Builtins::Generate_Adaptor(&masm, builtin_address, exit_frame_type);
+ CodeDesc desc;
+ masm.GetCode(&desc);
+ Handle<Code> code =
+ isolate->factory()->NewCode(desc, flags, masm.CodeObject());
+ PostBuildProfileAndTracing(isolate, *code, name);
+ return *code;
+}
+
+// Builder for builtins implemented in TurboFan with JS linkage.
+Code* BuildWithCodeStubAssemblerJS(Isolate* isolate,
+ CodeAssemblerGenerator generator, int argc,
+ Code::Flags flags, const char* name) {
+ HandleScope scope(isolate);
+ Zone zone(isolate->allocator(), ZONE_NAME);
+ const int argc_with_recv =
+ (argc == SharedFunctionInfo::kDontAdaptArgumentsSentinel) ? 0 : argc + 1;
+ compiler::CodeAssemblerState state(isolate, &zone, argc_with_recv, flags,
+ name);
+ generator(&state);
+ Handle<Code> code = compiler::CodeAssembler::GenerateCode(&state);
+ PostBuildProfileAndTracing(isolate, *code, name);
+ return *code;
+}
+
+// Builder for builtins implemented in TurboFan with CallStub linkage.
+Code* BuildWithCodeStubAssemblerCS(Isolate* isolate,
+ CodeAssemblerGenerator generator,
+ CallDescriptors::Key interface_descriptor,
+ Code::Flags flags, const char* name,
+ int result_size) {
+ HandleScope scope(isolate);
+ Zone zone(isolate->allocator(), ZONE_NAME);
+ // The interface descriptor with given key must be initialized at this point
+ // and this construction just queries the details from the descriptors table.
+ CallInterfaceDescriptor descriptor(isolate, interface_descriptor);
+ // Ensure descriptor is already initialized.
+ DCHECK_LE(0, descriptor.GetRegisterParameterCount());
+ compiler::CodeAssemblerState state(isolate, &zone, descriptor, flags, name,
+ result_size);
+ generator(&state);
+ Handle<Code> code = compiler::CodeAssembler::GenerateCode(&state);
+ PostBuildProfileAndTracing(isolate, *code, name);
+ return *code;
+}
+} // anonymous namespace
+
+void Builtins::SetUp(Isolate* isolate, bool create_heap_objects) {
+ DCHECK(!initialized_);
+
+ // Create a scope for the handles in the builtins.
+ HandleScope scope(isolate);
+
+ if (create_heap_objects) {
+ int index = 0;
+ const Code::Flags kBuiltinFlags = Code::ComputeFlags(Code::BUILTIN);
+ Code* code;
+#define BUILD_CPP(Name) \
+ code = BuildAdaptor(isolate, FUNCTION_ADDR(Builtin_##Name), BUILTIN_EXIT, \
+ kBuiltinFlags, #Name); \
+ builtins_[index++] = code;
+#define BUILD_API(Name) \
+ code = BuildAdaptor(isolate, FUNCTION_ADDR(Builtin_##Name), EXIT, \
+ kBuiltinFlags, #Name); \
+ builtins_[index++] = code;
+#define BUILD_TFJ(Name, Argc, ...) \
+ code = BuildWithCodeStubAssemblerJS(isolate, &Generate_##Name, Argc, \
+ kBuiltinFlags, #Name); \
+ builtins_[index++] = code;
+#define BUILD_TFS(Name, InterfaceDescriptor, result_size) \
+ { InterfaceDescriptor##Descriptor descriptor(isolate); } \
+ code = BuildWithCodeStubAssemblerCS(isolate, &Generate_##Name, \
+ CallDescriptors::InterfaceDescriptor, \
+ kBuiltinFlags, #Name, result_size); \
+ builtins_[index++] = code;
+#define BUILD_TFH(Name, Kind, Extra, InterfaceDescriptor) \
+ { InterfaceDescriptor##Descriptor descriptor(isolate); } \
+ /* Return size for IC builtins/handlers is always 1. */ \
+ code = BuildWithCodeStubAssemblerCS( \
+ isolate, &Generate_##Name, CallDescriptors::InterfaceDescriptor, \
+ Code::ComputeFlags(Code::Kind, Extra), #Name, 1); \
+ builtins_[index++] = code;
+#define BUILD_ASM(Name) \
+ code = \
+ BuildWithMacroAssembler(isolate, Generate_##Name, kBuiltinFlags, #Name); \
+ builtins_[index++] = code;
+
+ BUILTIN_LIST(BUILD_CPP, BUILD_API, BUILD_TFJ, BUILD_TFS, BUILD_TFH,
+ BUILD_ASM, BUILD_ASM);
+
+#undef BUILD_CPP
+#undef BUILD_API
+#undef BUILD_TFJ
+#undef BUILD_TFS
+#undef BUILD_TFH
+#undef BUILD_ASM
+ CHECK_EQ(builtin_count, index);
+ for (int i = 0; i < builtin_count; i++) {
+ Code::cast(builtins_[i])->set_builtin_index(i);
+ }
+
+#define SET_PROMISE_REJECTION_PREDICTION(Name) \
+ Code::cast(builtins_[k##Name])->set_is_promise_rejection(true);
+
+ BUILTIN_PROMISE_REJECTION_PREDICTION_LIST(SET_PROMISE_REJECTION_PREDICTION)
+#undef SET_PROMISE_REJECTION_PREDICTION
+
+#define SET_EXCEPTION_CAUGHT_PREDICTION(Name) \
+ Code::cast(builtins_[k##Name])->set_is_exception_caught(true);
+
+ BUILTIN_EXCEPTION_CAUGHT_PREDICTION_LIST(SET_EXCEPTION_CAUGHT_PREDICTION)
+#undef SET_EXCEPTION_CAUGHT_PREDICTION
+
+#define SET_CODE_NON_TAGGED_PARAMS(Name) \
+ Code::cast(builtins_[k##Name])->set_has_tagged_params(false);
+ BUILTINS_WITH_UNTAGGED_PARAMS(SET_CODE_NON_TAGGED_PARAMS)
+#undef SET_CODE_NON_TAGGED_PARAMS
+ }
+
+ // Mark as initialized.
+ initialized_ = true;
+}
+
void Builtins::TearDown() { initialized_ = false; }
void Builtins::IterateBuiltins(ObjectVisitor* v) {
« no previous file with comments | « src/builtins/builtins.h ('k') | src/builtins/builtins-call.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698