[arm64] Consider branch when skipping a veener pool

victorgomes · V8 LUCI CQ · commit 53786ab8bc42 · 2025-07-16T08:04:20.000-07:00
... in InstructionAccurateScope. Bug: 431828026 Change-Id: Iee98fe6172f2bd0da65d5acf3656294be8449705 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/6759971 Auto-Submit: Victor Gomes <victorgomes@chromium.org> Reviewed-by: Leszek Swirski <leszeks@chromium.org> Commit-Queue: Victor Gomes <victorgomes@chromium.org> Commit-Queue: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/main@{#101462}
diff --git a/src/codegen/arm64/macro-assembler-arm64.h b/src/codegen/arm64/macro-assembler-arm64.h
@@ -2508,7 +2508,9 @@ class V8_NODISCARD InstructionAccurateScope {
 #endif
   {
     DCHECK_GT(count, 0);
-    masm_->CheckVeneerPool(false, true, count * kInstrSize);
+    // We include the branch instruction in the veneer distance margin if we
+    // need to emit a veneer pool.
+    masm_->CheckVeneerPool(false, true, (count + 1) * kInstrSize);
     masm_->StartBlockVeneerPool();
 #ifdef DEBUG
     masm_->bind(&start_);
