Bug 354274

Summary: arm: unhandled instruction: 0xEBAD 0x0AC1 (sub.w sl, sp, r1, lsl #3)
Product: [Developer tools] valgrind Reporter: dimitry <dimitry>
Component: vexAssignee: Julian Seward <jseward>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version First Reported In: 3.10 SVN   
Target Milestone: ---   
Platform: Android   
OS: Other   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description dimitry 2015-10-23 18:33:34 UTC
This patch fixed the problem:

diff --git a/VEX/priv/guest_arm_toIR.c b/VEX/priv/guest_arm_toIR.c
index d2bcf16..54ccd07 100644
--- a/VEX/priv/guest_arm_toIR.c
+++ b/VEX/priv/guest_arm_toIR.c
@@ -19671,10 +19671,10 @@ DisResult disInstr_THUMB_WRK (
           && rD != 15 && rN == 13 && imm5 <= 3 && how == 0) {
          valid = True;
       }
-      /* also allow "sub.w reg, sp, reg   w/ no shift
+      /* also allow "sub.w reg, sp, reg   lsl #N for N=0,1,2 or 3
          (T1) "SUB (SP minus register) */
       if (!valid && INSN0(8,5) == BITS4(1,1,0,1) // sub
-          && rD != 15 && rN == 13 && imm5 == 0 && how == 0) {
+          && rD != 15 && rN == 13 && imm5 <= 3 && how == 0) {
          valid = True;
       }
       if (valid) {
Comment 1 Julian Seward 2016-10-06 05:26:49 UTC
Committed as vex r3257, and will be in 3.12.0.  Thanks for the patch.