From 73665cc31f47264b70b4c9755ee2be1712b4f532 Mon Sep 17 00:00:00 2001 From: Kunal Pathak Date: Thu, 14 Apr 2022 11:07:42 -0700 Subject: [PATCH] Pass the argument to formatting function (#67996) * pass block number * jit formatting * fix another issue * Revert "fix another issue" This reverts commit 637c340ad92af810a637e4867fbd12403b40661f. * fix some other formatting issues --- src/coreclr/jit/compiler.cpp | 2 +- src/coreclr/jit/fgbasic.cpp | 3 ++- src/coreclr/jit/lower.cpp | 4 ++-- 3 files changed, 5 insertions(+), 4 deletions(-) diff --git a/src/coreclr/jit/compiler.cpp b/src/coreclr/jit/compiler.cpp index ea036fc7b1da9..907c40f27c17e 100644 --- a/src/coreclr/jit/compiler.cpp +++ b/src/coreclr/jit/compiler.cpp @@ -9134,7 +9134,7 @@ Compiler::LoopDsc* dFindLoop(unsigned loopNum) if (loopNum >= comp->optLoopCount) { - printf("loopNum %u out of range\n"); + printf("loopNum %u out of range\n", loopNum); return nullptr; } diff --git a/src/coreclr/jit/fgbasic.cpp b/src/coreclr/jit/fgbasic.cpp index 632978641b9ee..f2143ff0f5adf 100644 --- a/src/coreclr/jit/fgbasic.cpp +++ b/src/coreclr/jit/fgbasic.cpp @@ -3589,7 +3589,8 @@ void Compiler::fgCheckForLoopsInHandlers() { if (blk->bbFlags & BBF_BACKWARD_JUMP_TARGET) { - JITDUMP("\nHander block " FMT_BB "is backward jump target; can't have patchpoints in this method\n"); + JITDUMP("\nHander block " FMT_BB "is backward jump target; can't have patchpoints in this method\n", + blk->bbNum); compHasBackwardJumpInHandler = true; break; } diff --git a/src/coreclr/jit/lower.cpp b/src/coreclr/jit/lower.cpp index f483b748bd47a..f91c1306758c7 100644 --- a/src/coreclr/jit/lower.cpp +++ b/src/coreclr/jit/lower.cpp @@ -52,8 +52,8 @@ void Lowering::MakeSrcContained(GenTree* parentNode, GenTree* childNode) const if (!isSafeToContainMem) { - JITDUMP("** Unsafe mem containment of [%06u] in [%06u}, comp->dspTreeID(childNode), " - "comp->dspTreeID(parentNode)\n"); + JITDUMP("** Unsafe mem containment of [%06u] in [%06u}\n", comp->dspTreeID(childNode), + comp->dspTreeID(parentNode)); assert(isSafeToContainMem); } }