From: notaz <notasas@gmail.com>
Date: Sun, 8 Sep 2013 01:40:40 +0000 (+0300)
Subject: avoid using msr
X-Git-Url: https://notaz.gp2x.de/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=cfe17eee41a12da11f69660c2764813e94d871ea;p=cyclone68000.git

avoid using msr

causes pipeline flush on newer ARMs, and we don't want that
---

diff --git a/OpArith.cpp b/OpArith.cpp
index d762049..ca279da 100644
--- a/OpArith.cpp
+++ b/OpArith.cpp
@@ -373,7 +373,7 @@ int GetXBit(int subtract)
   ot(";@ Get X bit:\n");
   ot("  ldr r2,[r7,#0x4c]\n");
   if (subtract) ot("  mvn r2,r2 ;@ Invert it\n");
-  ot("  msr cpsr_flg,r2 ;@ Get into Carry\n");
+  ot("  tst r2,r2,lsl #3 ;@ Get into Carry\n");
   ot("\n");
   return 0;
 }
diff --git a/OpBranch.cpp b/OpBranch.cpp
index fb4f561..051c373 100644
--- a/OpBranch.cpp
+++ b/OpBranch.cpp
@@ -312,6 +312,7 @@ static const char * const Cond[16]=
 // Emit a Dbra opcode, 0101cccc 11001nnn vv
 int OpDbra(int op)
 {
+  const char *cond;
   int use=0;
   int cc=0;
 
@@ -321,25 +322,12 @@ int OpDbra(int op)
   if (op!=use) { OpUse(op,use); return 0; } // Use existing handler
   OpStart(op);
 
-  switch (cc)
+  if (cc>=2)
   {
-    case 0: // T
-    case 1: // F
-      break;
-    case 2: // hi
-      ot("  tst r10,#0x60000000 ;@ hi: !C && !Z\n");
-      ot("  beq DbraTrue\n\n");
-      break;
-    case 3: // ls
-      ot("  tst r10,#0x60000000 ;@ ls: C || Z\n");
-      ot("  bne DbraTrue\n\n");
-      break;
-    default:
-      ot(";@ Is the condition true?\n");
-      ot("  msr cpsr_flg,r10 ;@ ARM flags = 68000 flags\n");
-      ot(";@ If so, don't dbra\n");
-      ot("  b%s DbraTrue\n\n",Cond[cc]);
-      break;
+    ot(";@ Is the condition true?\n");
+    cond=TestCond(cc);
+    ot(";@ If so, don't dbra\n");
+    ot("  b%s DbraTrue\n\n",cond);
   }
 
   if (cc!=0)
@@ -419,6 +407,7 @@ int OpBranch(int op)
   int offset=0;
   int cc=0;
   const char *asr_r11="";
+  const char *cond;
   int pc_reg=0;
 
   offset=(char)(op&0xff);
@@ -436,24 +425,10 @@ int OpBranch(int op)
   OpStart(op,size?0x10:0);
   Cycles=10; // Assume branch taken
 
-  switch (cc)
+  if (cc>=2)
   {
-    case 0: // T
-    case 1: // F
-      break;
-    case 2: // hi
-      ot("  tst r10,#0x60000000 ;@ hi: !C && !Z\n");
-      ot("  bne BccDontBranch%i\n\n",8<<size);
-      break;
-    case 3: // ls
-      ot("  tst r10,#0x60000000 ;@ ls: C || Z\n");
-      ot("  beq BccDontBranch%i\n\n",8<<size);
-      break;
-    default:
-      ot(";@ Is the condition true?\n");
-      ot("  msr cpsr_flg,r10 ;@ ARM flags = 68000 flags\n");
-      ot("  b%s BccDontBranch%i\n\n",Cond[cc^1],8<<size);
-      break;
+    cond=TestCond(cc,1);
+    ot("  b%s BccDontBranch%i\n\n",cond,8<<size);
   }
 
   if (size) 
diff --git a/OpLogic.cpp b/OpLogic.cpp
index d4d9b13..19ec569 100644
--- a/OpLogic.cpp
+++ b/OpLogic.cpp
@@ -12,6 +12,84 @@
 
 #include "app.h"
 
+// trashes r0
+const char *TestCond(int m68k_cc, int invert)
+{
+  const char *cond="";
+  const char *icond="";
+
+  // ARM: NZCV
+  switch (m68k_cc)
+  {
+    case 0x00: // T
+    case 0x01: // F
+      break;
+    case 0x02: // hi
+      ot("  tst r10,#0x60000000 ;@ hi: !C && !Z\n");
+      cond="eq", icond="ne";
+      break;
+    case 0x03: // ls
+      ot("  tst r10,#0x60000000 ;@ ls: C || Z\n");
+      cond="ne", icond="eq";
+      break;
+    case 0x04: // cc
+      ot("  tst r10,#0x20000000 ;@ cc: !C\n");
+      cond="eq", icond="ne";
+      break;
+    case 0x05: // cs
+      ot("  tst r10,#0x20000000 ;@ cs: C\n");
+      cond="ne", icond="eq";
+      break;
+    case 0x06: // ne
+      ot("  tst r10,#0x40000000 ;@ ne: !Z\n");
+      cond="eq", icond="ne";
+      break;
+    case 0x07: // eq
+      ot("  tst r10,#0x40000000 ;@ eq: Z\n");
+      cond="ne", icond="eq";
+      break;
+    case 0x08: // vc
+      ot("  tst r10,#0x10000000 ;@ vc: !V\n");
+      cond="eq", icond="ne";
+      break;
+    case 0x09: // vs
+      ot("  tst r10,#0x10000000 ;@ vs: V\n");
+      cond="ne", icond="eq";
+      break;
+    case 0x0a: // pl
+      ot("  tst r10,r10 ;@ pl: !N\n");
+      cond="pl", icond="mi";
+      break;
+    case 0x0b: // mi
+      ot("  tst r10,r10 ;@ mi: N\n");
+      cond="mi", icond="pl";
+      break;
+    case 0x0c: // ge
+      ot("  teq r10,r10,lsl #3 ;@ ge: N == V\n");
+      cond="pl", icond="mi";
+      break;
+    case 0x0d: // lt
+      ot("  teq r10,r10,lsl #3 ;@ lt: N != V\n");
+      cond="mi", icond="pl";
+      break;
+    case 0x0e: // gt
+      ot("  eor r0,r10,r10,lsl #3 ;@ gt: !Z && N == V\n");
+      ot("  orrs r0,r10,lsl #1\n");
+      cond="pl", icond="mi";
+      break;
+    case 0x0f: // le
+      ot("  eor r0,r10,r10,lsl #3 ;@ le: Z || N != V\n");
+      ot("  orrs r0,r10,lsl #1\n");
+      cond="mi", icond="pl";
+      break;
+    default:
+      printf("invalid m68k_cc: %x\n", m68k_cc);
+      exit(1);
+      break;
+  }
+  return invert?icond:cond;
+}
+
 // --------------------- Opcodes 0x0100+ ---------------------
 // Emit a Btst (Register) opcode 0000nnn1 ttaaaaaa
 int OpBtstReg(int op)
@@ -318,11 +396,7 @@ int OpSet(int op)
 {
   int cc=0,ea=0;
   int size=0,use=0,changed_cycles=0;
-  static const char * const cond[16]=
-  {
-    "al","", "hi","ls","cc","cs","ne","eq",
-    "vc","vs","pl","mi","ge","lt","gt","le"
-  };
+  const char *cond;
 
   cc=(op>>8)&15;
   ea=op&0x003f;
@@ -339,32 +413,20 @@ int OpSet(int op)
   OpStart(op,ea,0,changed_cycles); Cycles=8;
   if (ea<8) Cycles=4;
 
-  if (cc)
-    ot("  mov r1,#0\n");
-
   switch (cc)
   {
-    case 0: // T
+    case 0x00: // T
       ot("  mvn r1,#0\n");
       if (ea<8) Cycles+=2;
       break;
-    case 1: // F
-      break;
-    case 2: // hi
-      ot("  tst r10,#0x60000000 ;@ hi: !C && !Z\n");
-      ot("  mvneq r1,r1\n");
-      if (ea<8) ot("  subeq r5,r5,#2 ;@ Extra cycles\n");
-      break;
-    case 3: // ls
-      ot("  tst r10,#0x60000000 ;@ ls: C || Z\n");
-      ot("  mvnne r1,r1\n");
-      if (ea<8) ot("  subne r5,r5,#2 ;@ Extra cycles\n");
+    case 0x01: // F
+      ot("  mov r1,#0\n");
       break;
     default:
-      ot(";@ Is the condition true?\n");
-      ot("  msr cpsr_flg,r10 ;@ ARM flags = 68000 flags\n");
-      ot("  mvn%s r1,r1\n",cond[cc]);
-      if (ea<8) ot("  sub%s r5,r5,#2 ;@ Extra cycles\n",cond[cc]);
+      ot("  mov r1,#0\n");
+      cond=TestCond(cc);
+      ot("  mvn%s r1,#0\n",cond);
+      if (ea<8) ot("  sub%s r5,r5,#2 ;@ Extra cycles\n",cond);
       break;
   }
 
diff --git a/app.h b/app.h
index 3528b7b..4a23630 100644
--- a/app.h
+++ b/app.h
@@ -102,6 +102,7 @@ int OpSet(int op);
 int OpAsr(int op);
 int OpAsrEa(int op);
 int OpTas(int op, int gen_special=0);
+const char *TestCond(int m68k_cc, int invert=0);
 
 // OpMove.cpp
 int OpMove(int op);
diff --git a/tools/idle.s b/tools/idle.s
index e970d2e..e55b5f1 100644
--- a/tools/idle.s
+++ b/tools/idle.s
@@ -104,15 +104,15 @@ idle_bra:
     b       Op6002
 
 idle_bne:
-    msr     cpsr_flg, r10
-    movne   r5, #2                @ 2 is intentional due to strange timing issues
-    inc_counter ne
+    tst     r10, #0x40000000      @ Z set?
+    moveq   r5, #2                @ 2 is intentional due to strange timing issues
+    inc_counter eq
     b       Op6602
 
 idle_beq:
-    msr     cpsr_flg, r10 ;@ ARM flags = 68000 flags
-    moveq   r5, #2
-    inc_counter eq
+    tst     r10, #0x40000000      @ Z set?
+    movne   r5, #2
+    inc_counter ne
     b       Op6702