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

Unified Diff: src/builtins/builtins.cc

Issue 2810363003: [builtins] Introduce DeleteProperty builtin (Closed)
Patch Set: fix CallableFor not to create handles 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 | « no previous file | src/builtins/builtins-definitions.h » ('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 8fb4844e377f0782dbf7ecd869eccd88bb590c20..45a4842bb3b349384394167bf0f5c6144df60b6f 100644
--- a/src/builtins/builtins.cc
+++ b/src/builtins/builtins.cc
@@ -116,11 +116,11 @@ Handle<Code> Builtins::OrdinaryToPrimitive(OrdinaryToPrimitiveHint hint) {
// static
Callable Builtins::CallableFor(Isolate* isolate, Name name) {
switch (name) {
-#define CASE(Name, ...) \
- case k##Name: { \
- Handle<Code> code(Code::cast(isolate->builtins()->builtins_[name])); \
- auto descriptor = Builtin_##Name##_InterfaceDescriptor(isolate); \
- return Callable(code, descriptor); \
+#define CASE(Name, ...) \
+ case k##Name: { \
+ Handle<Code> code = isolate->builtins()->Name(); \
+ auto descriptor = Builtin_##Name##_InterfaceDescriptor(isolate); \
+ return Callable(code, descriptor); \
}
BUILTIN_LIST(IGNORE_BUILTIN, IGNORE_BUILTIN, IGNORE_BUILTIN, CASE, CASE,
CASE, IGNORE_BUILTIN, IGNORE_BUILTIN)
« no previous file with comments | « no previous file | src/builtins/builtins-definitions.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698