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

Unified Diff: src/code-stub-assembler.cc

Issue 2575313002: [promisehook] Implement PromiseHook (Closed)
Patch Set: remove promise_hook_enabled_ Created 4 years 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: src/code-stub-assembler.cc
diff --git a/src/code-stub-assembler.cc b/src/code-stub-assembler.cc
index 45c3f7e3d0e91f5558c11ed79618a0ea3b7271e5..3b0d0a38c670d74506065b66167e423e1d484db3 100644
--- a/src/code-stub-assembler.cc
+++ b/src/code-stub-assembler.cc
@@ -8242,11 +8242,10 @@ Node* CodeStubAssembler::IsDebugActive() {
}
Node* CodeStubAssembler::IsPromiseHookEnabled() {
- Node* const is_promisehook_enabled =
- Load(MachineType::Uint8(),
- ExternalConstant(
- ExternalReference::is_promisehook_enabled_address(isolate())));
- return Word32NotEqual(is_promisehook_enabled, Int32Constant(0));
+ Node* const promise_hook = Load(
+ MachineType::Uint8(),
adamk 2016/12/15 15:43:58 Probably this needs to change now that it's a poin
gsathya 2016/12/15 16:32:06 Doh, ofc. Sorry for being sloppy!
+ ExternalConstant(ExternalReference::promise_hook_address(isolate())));
+ return Word32NotEqual(promise_hook, Int32Constant(0));
}
Node* CodeStubAssembler::AllocateJSPromise(Node* context) {

Powered by Google App Engine
This is Rietveld 408576698