Commit b1c1c46
src: remove V8_FT_ADAPTOR for V8 update
V8 has removed argument adapter frames. This commit removes them
from the Node.js codebase as well.
PR-URL: #37587
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>1 parent 9843361 commit b1c1c46
2 files changed
Lines changed: 0 additions & 19 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -41,7 +41,6 @@ | |||
| 41 | 41 | #define V8_FT_OPTIMIZED V8DBG_FRAMETYPE_OPTIMIZEDFRAME | |
| 42 | 42 | #define V8_FT_INTERNAL V8DBG_FRAMETYPE_INTERNALFRAME | |
| 43 | 43 | #define V8_FT_CONSTRUCT V8DBG_FRAMETYPE_CONSTRUCTFRAME | |
| 44 | - #define V8_FT_ADAPTOR V8DBG_FRAMETYPE_ARGUMENTSADAPTORFRAME | ||
| 45 | 44 | #define V8_FT_STUB V8DBG_FRAMETYPE_STUBFRAME | |
| 46 | 45 | ||
| 47 | 46 | /* Identification masks and tags */ | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -392,24 +392,6 @@ dtrace:helper:ustack: | |||
| 392 | 392 | this->ii = 0; | |
| 393 | 393 | } | |
| 394 | 394 | ||
| 395 | - /* | ||
| 396 | - * Like V8, we first check if we've got an ArgumentsAdaptorFrame. We've got | ||
| 397 | - * nothing to add for such frames, so we bail out quickly. | ||
| 398 | - */ | ||
| 399 | - dtrace:helper:ustack: | ||
| 400 | - { | ||
| 401 | - this->ctx = COPYIN_PTR(this->fp + V8_OFF_FP_CONTEXT); | ||
| 402 | - } | ||
| 403 | - | ||
| 404 | - dtrace:helper:ustack: | ||
| 405 | - /IS_SMI(this->ctx) && SMI_VALUE(this->ctx) == V8_FT_ADAPTOR/ | ||
| 406 | - { | ||
| 407 | - this->done = 1; | ||
| 408 | - APPEND_CHR8('<','<',' ','a','d','a','p','t'); | ||
| 409 | - APPEND_CHR8('o','r',' ','>','>','\0','\0','\0'); | ||
| 410 | - stringof(this->buf); | ||
| 411 | - } | ||
| 412 | - | ||
| 413 | 395 | /* | |
| 414 | 396 | * Check for other common frame types for which we also have nothing to add. | |
| 415 | 397 | */ | |
0 commit comments