AuroraMiddleware/v8
1
0
Fork 0
Code Issues 2 Pull Requests Releases Wiki Activity

Revert "[riscv] Remove unnecessary unbound label count"

Browse Source 
This reverts commit a6c2b39080.

Reason for revert: Failed tests

Original change's description:
> [riscv] Remove unnecessary unbound label count
>
> The bind_to function doesn't link branch long to trampoline, so it doesn't need to add  unbound_labels_count_.
>
> Change-Id: I2e3861a38eb65c285f19accb12bccb9f4c9fcfb1
> Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/4133426
> Auto-Submit: Yahan Lu <yahan@iscas.ac.cn>
> Reviewed-by: ji qiu <qiuji@iscas.ac.cn>
> Commit-Queue: ji qiu <qiuji@iscas.ac.cn>
> Cr-Commit-Position: refs/heads/main@{#85103}

Change-Id: I651762d71a8e86bbe76a10224a63433cdacfadfe
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/4136999
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Auto-Submit: Yahan Lu <yahan@iscas.ac.cn>
Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Cr-Commit-Position: refs/heads/main@{#85108}
This commit is contained in:
Yahan Lu 2023-01-05 07:35:21 +00:00 committed by V8 LUCI CQ
parent 3094c4002b
commit fcae4c1383
1 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/codegen/riscv/assembler-riscv.cc
View File

@ -782,6 +782,10 @@ int32_t Assembler::branch_long_offset(Label* L) {
L->link_to(pc_offset());
} else {
L->link_to(pc_offset());
if (!trampoline_emitted_) {
unbound_labels_count_++;
next_buffer_check_ -= kTrampolineSlotsSize;
}
DEBUG_PRINTF("\tstarted link\n");
return kEndOfJumpChain;
}