tor-browser

commit f9cd51be543e96175b5c19e0e69b2ec08326c93f
parent fadf320e1749ba3c29f1c29e28ec74686e9fd2ef
Author: Rong "Mantle" Bao <webmaster@csmantle.top>
Date:   Thu, 27 Nov 2025 11:57:06 +0000

Bug 1996840 - [riscv64] Part 6: (Drive-by) Improve debug output for branch linking. r=nbp

Differential Revision: https://phabricator.services.mozilla.com/D271753

Diffstat:
M
js/src/jit/riscv64/Assembler-riscv64.cpp
 | 
13
+++++++------

1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a/js/src/jit/riscv64/Assembler-riscv64.cpp b/js/src/jit/riscv64/Assembler-riscv64.cpp
@@ -1027,8 +1027,8 @@ uint32_t Assembler::next_link(Label* L, bool is_internal) {
 }
 
 void Assembler::bind(Label* label, BufferOffset boff) {
-  JitSpew(JitSpew_Codegen, ".set Llabel %p %d", label, currentOffset());
-  DEBUG_PRINTF(".set Llabel %p\n", label);
+  JitSpew(JitSpew_Codegen, ".set Llabel %p %u", label, currentOffset());
+  DEBUG_PRINTF(".set Llabel %p %u\n", label, currentOffset());
   // If our caller didn't give us an explicit target to bind to
   // then we want to bind to the location of the next instruction
   BufferOffset dest = boff.assigned() ? boff : nextOffset();
@@ -1047,10 +1047,11 @@ void Assembler::bind(Label* label, BufferOffset boff) {
       int fixup_pos = b.getOffset();
       int dist = dest.getOffset() - fixup_pos;
       next = next_link(label, false);
-      DEBUG_PRINTF("\t%p fixup: %d next: %d\n", label, fixup_pos, next);
-      DEBUG_PRINTF("\t   fixup: %d dest: %d dist: %d %d %d\n", fixup_pos,
-                   dest.getOffset(), dist, nextOffset().getOffset(),
-                   currentOffset());
+      DEBUG_PRINTF(
+          "\t%p fixup: %d next: %u dest: %d dist: %d nextOffset: %d "
+          "currOffset: %d\n",
+          label, fixup_pos, next, dest.getOffset(), dist,
+          nextOffset().getOffset(), currentOffset());
       Instr instr = editSrc(b)->InstructionBits();
       if (IsBranch(instr)) {
         if (!is_intn(dist, kBranchOffsetBits)) {