From a639a807b392274efc551b56da57140d55fae67d Mon Sep 17 00:00:00 2001 From: Carl Love Date: Fri, 5 Jun 2015 18:52:57 +0000 Subject: [PATCH] Opps, missed a change in the previous patch. Forgot to remove the format specifier. The dcbt and dcbtst instructions provide a non-zero hint that describes a block or data stream to which a program may perform a Store acces, or indicates the expected use. The field bits[25:21] (bits 6:10 in the IBM numbering) in the instruction provide the hint. Valgrind checks that these bits are non-zero. Unfortunately, the test was being applied to other instructions such as the dcbf instruction causing it to fail when the field was equal to zero. This patch removes the check that was being incorrectly applied to all of the instructions. The valgrind bugzilla for this issue is 348334. git-svn-id: svn://svn.valgrind.org/vex/trunk@3149 --- VEX/priv/guest_ppc_toIR.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/VEX/priv/guest_ppc_toIR.c b/VEX/priv/guest_ppc_toIR.c index 103ed368c8..e4d2209fb2 100644 --- a/VEX/priv/guest_ppc_toIR.c +++ b/VEX/priv/guest_ppc_toIR.c @@ -7463,7 +7463,7 @@ static Bool dis_cache_manage ( UInt theInstr, } if (opc1 != 0x1F || b0 != 0) { - if (0) vex_printf("dis_cache_manage %d %d %d\n", + if (0) vex_printf("dis_cache_manage %d %d\n", (Int)opc1, (Int)b0); vex_printf("dis_cache_manage(ppc)(opc1|b0)\n"); return False; -- 2.47.2