]> git.ipfire.org Git - thirdparty/valgrind.git/commitdiff
Cleanup backend: var name chages like src1,2 -> srcL,R etc
authorCerion Armour-Brown <cerion@valgrind.org>
Tue, 15 Mar 2005 16:33:38 +0000 (16:33 +0000)
committerCerion Armour-Brown <cerion@valgrind.org>
Tue, 15 Mar 2005 16:33:38 +0000 (16:33 +0000)
git-svn-id: svn://svn.valgrind.org/vex/trunk@1033

VEX/priv/host-ppc32/hdefs.c
VEX/priv/host-ppc32/hdefs.h
VEX/priv/host-ppc32/isel.c

index fac2c41a9bedcec0dbbdad754571a49d1c393056..166670787be3a3478bff7d440622067ebe74c5c0 100644 (file)
@@ -450,25 +450,25 @@ HChar* showPPC32CmpOp ( PPC32CmpOp op ) {
 //..    }
 //.. }
 
-PPC32Instr* PPC32Instr_Alu32 ( PPC32AluOp op, HReg dst, HReg src1, PPC32RI* src2 ) {
+PPC32Instr* PPC32Instr_Alu32 ( PPC32AluOp op, HReg dst, HReg srcL, PPC32RI* srcR ) {
    PPC32Instr* i     = LibVEX_Alloc(sizeof(PPC32Instr));
-   if (src2->tag == Pri_Imm)
-      vassert(src2->Pri.Imm.imm32 < 0x10000);
+   if (srcR->tag == Pri_Imm)
+      vassert(srcR->Pri.Imm.imm32 < 0x10000);
    i->tag            = Pin_Alu32;
    i->Pin.Alu32.op   = op;
    i->Pin.Alu32.dst  = dst;
-   i->Pin.Alu32.src1 = src1;
-   i->Pin.Alu32.src2 = src2;
+   i->Pin.Alu32.srcL = srcL;
+   i->Pin.Alu32.srcR = srcR;
    return i;
 }
-PPC32Instr* PPC32Instr_Sub32 ( HReg dst, PPC32RI* src1, HReg src2 ) {
+PPC32Instr* PPC32Instr_Sub32 ( HReg dst, PPC32RI* srcL, HReg srcR ) {
    PPC32Instr* i     = LibVEX_Alloc(sizeof(PPC32Instr));
-   if (src1->tag == Pri_Imm)
-      vassert(src1->Pri.Imm.imm32 < 0x10000);
+   if (srcL->tag == Pri_Imm)
+      vassert(srcL->Pri.Imm.imm32 < 0x10000);
    i->tag            = Pin_Sub32;
    i->Pin.Sub32.dst  = dst;
-   i->Pin.Sub32.src1 = src1;
-   i->Pin.Sub32.src2 = src2;
+   i->Pin.Sub32.srcL = srcL;
+   i->Pin.Sub32.srcR = srcR;
    return i;
 }
 PPC32Instr* PPC32Instr_Sh32 ( PPC32ShiftOp op, HReg dst, HReg src, PPC32RI* shft ) {
@@ -484,15 +484,15 @@ PPC32Instr* PPC32Instr_Sh32 ( PPC32ShiftOp op, HReg dst, HReg src, PPC32RI* shft
    i->Pin.Sh32.shft = shft;
    return i;
 }
-PPC32Instr* PPC32Instr_Cmp32 ( PPC32CmpOp op, UInt crfD, HReg src1, PPC32RI* src2 ) {
+PPC32Instr* PPC32Instr_Cmp32 ( PPC32CmpOp op, UInt crfD, HReg srcL, PPC32RI* srcR ) {
    PPC32Instr* i     = LibVEX_Alloc(sizeof(PPC32Instr));
-   if (src2->tag == Pri_Imm)
-      vassert(src2->Pri.Imm.imm32 < 0x10000);
+   if (srcR->tag == Pri_Imm)
+      vassert(srcR->Pri.Imm.imm32 < 0x10000);
    i->tag            = Pin_Cmp32;
    i->Pin.Cmp32.op   = op;
    i->Pin.Cmp32.crfD = crfD;
-   i->Pin.Cmp32.src1 = src1;
-   i->Pin.Cmp32.src2 = src2;
+   i->Pin.Cmp32.srcL = srcL;
+   i->Pin.Cmp32.srcR = srcR;
    return i;
 }
 PPC32Instr* PPC32Instr_Unary32  ( PPC32UnaryOp op, HReg dst, HReg src ) {
@@ -504,31 +504,31 @@ PPC32Instr* PPC32Instr_Unary32  ( PPC32UnaryOp op, HReg dst, HReg src ) {
    return i;
 }
 PPC32Instr* PPC32Instr_MulL ( Bool syned, Bool word, HReg dst,
-                              HReg src1, PPC32RI* src2 ) {
+                              HReg srcL, PPC32RI* srcR ) {
    PPC32Instr* i     = LibVEX_Alloc(sizeof(PPC32Instr));
    if (word == 1) { // high
-      vassert(src2->tag == Pri_Reg);
+      vassert(srcR->tag == Pri_Reg);
    } else {         // low
-      if (src2->tag == Pri_Imm) {
+      if (srcR->tag == Pri_Imm) {
          vassert(syned == True);
-         vassert(src2->Pri.Imm.imm32 < 0x10000);
+         vassert(srcR->Pri.Imm.imm32 < 0x10000);
       }
    }
    i->tag            = Pin_MulL;
    i->Pin.MulL.syned = syned;
    i->Pin.MulL.word  = word;
    i->Pin.MulL.dst   = dst;
-   i->Pin.MulL.src1  = src1;
-   i->Pin.MulL.src2  = src2;
+   i->Pin.MulL.srcL  = srcL;
+   i->Pin.MulL.srcR  = srcR;
    return i;
 }
-PPC32Instr* PPC32Instr_Div ( Bool syned, HReg dst, HReg src1, HReg src2 ) {
+PPC32Instr* PPC32Instr_Div ( Bool syned, HReg dst, HReg srcL, HReg srcR ) {
    PPC32Instr* i      = LibVEX_Alloc(sizeof(PPC32Instr));
    i->tag             = Pin_Div;
    i->Pin.Div.syned   = syned;
    i->Pin.Div.dst     = dst;
-   i->Pin.Div.src1    = src1;
-   i->Pin.Div.src2    = src2;
+   i->Pin.Div.srcL    = srcL;
+   i->Pin.Div.srcR    = srcR;
    return i;
 }
 //.. X86Instr* X86Instr_Sh3232  ( X86ShiftOp op, UInt amt, HReg src, HReg dst ) {
@@ -733,48 +733,48 @@ void ppPPC32Instr ( PPC32Instr* i )
    switch (i->tag) {
    case Pin_Alu32:
       if (i->Pin.Alu32.op == Palu_OR &&     // or Rd,Rs,Rs == mr Rd,Rs
-          i->Pin.Alu32.src2->tag == Pri_Reg &&
-          i->Pin.Alu32.src2->Pri.Reg.reg == i->Pin.Alu32.src1) {
+          i->Pin.Alu32.srcR->tag == Pri_Reg &&
+          i->Pin.Alu32.srcR->Pri.Reg.reg == i->Pin.Alu32.srcL) {
          vex_printf("mr ");
          ppHRegPPC32(i->Pin.Alu32.dst);
          vex_printf(",");
-         ppHRegPPC32(i->Pin.Alu32.src1);
+         ppHRegPPC32(i->Pin.Alu32.srcL);
          return;
       }
       if (i->Pin.Alu32.op == Palu_ADD &&     // add rD,r0,val == li rD,val
-          i->Pin.Alu32.src1 == hregPPC32_GPR0() &&
-          i->Pin.Alu32.src2->tag == Pri_Imm) {
+          i->Pin.Alu32.srcL == hregPPC32_GPR0() &&
+          i->Pin.Alu32.srcR->tag == Pri_Imm) {
          vex_printf("li ");
          ppHRegPPC32(i->Pin.Alu32.dst);
          vex_printf(",");
-         ppPPC32RI(i->Pin.Alu32.src2);
+         ppPPC32RI(i->Pin.Alu32.srcR);
          return;
       }
       if (i->Pin.Alu32.op == Palu_AND &&     // 'andi.' - always has the '.'
-          i->Pin.Alu32.src2->tag == Pri_Imm) {
+          i->Pin.Alu32.srcR->tag == Pri_Imm) {
          vex_printf("andi. ");
          ppHRegPPC32(i->Pin.Alu32.dst);
          vex_printf(",");
-         ppHRegPPC32(i->Pin.Alu32.src1);
+         ppHRegPPC32(i->Pin.Alu32.srcL);
          vex_printf(",");
-         ppPPC32RI(i->Pin.Alu32.src2);
+         ppPPC32RI(i->Pin.Alu32.srcR);
          return;
       }
       vex_printf("%s%s ", showPPC32AluOp(i->Pin.Alu32.op),
-                 i->Pin.Alu32.src2->tag == Pri_Imm ? "i" : "" );
+                 i->Pin.Alu32.srcR->tag == Pri_Imm ? "i" : "" );
       ppHRegPPC32(i->Pin.Alu32.dst);
       vex_printf(",");
-      ppHRegPPC32(i->Pin.Alu32.src1);
+      ppHRegPPC32(i->Pin.Alu32.srcL);
       vex_printf(",");
-      ppPPC32RI(i->Pin.Alu32.src2);
+      ppPPC32RI(i->Pin.Alu32.srcR);
       return;
    case Pin_Sub32:
-      vex_printf("subf%s ", i->Pin.Sub32.src1->tag == Pri_Imm ? "ic" : "" );
+      vex_printf("subf%s ", i->Pin.Sub32.srcL->tag == Pri_Imm ? "ic" : "" );
       ppHRegPPC32(i->Pin.Sub32.dst);
       vex_printf(",");
-      ppHRegPPC32(i->Pin.Sub32.src2); // yes, order is right :-)
+      ppHRegPPC32(i->Pin.Sub32.srcR); // yes, order is right :-)
       vex_printf(",");
-      ppPPC32RI(i->Pin.Sub32.src1);
+      ppPPC32RI(i->Pin.Sub32.srcL);
       return;
    case Pin_Sh32:
       vex_printf("%s%s ", showPPC32ShiftOp(i->Pin.Sh32.op),
@@ -787,11 +787,11 @@ void ppPPC32Instr ( PPC32Instr* i )
       return;
    case Pin_Cmp32:
       vex_printf("%s%s %%crf%d,", showPPC32CmpOp(i->Pin.Cmp32.op),
-                 i->Pin.Cmp32.src2->tag == Pri_Imm ? "i" : "",
+                 i->Pin.Cmp32.srcR->tag == Pri_Imm ? "i" : "",
                  (7 - i->Pin.Cmp32.crfD));
-      ppHRegPPC32(i->Pin.Cmp32.src1);
+      ppHRegPPC32(i->Pin.Cmp32.srcL);
       vex_printf(",");
-      ppPPC32RI(i->Pin.Cmp32.src2);
+      ppPPC32RI(i->Pin.Cmp32.srcR);
       return;
    case Pin_Unary32:
       vex_printf("%s ", showPPC32UnaryOp(i->Pin.Unary32.op));
@@ -800,7 +800,7 @@ void ppPPC32Instr ( PPC32Instr* i )
       ppHRegPPC32(i->Pin.Unary32.src);
       return;
    case Pin_MulL:
-      if (i->Pin.MulL.src2->tag == Pri_Imm) {
+      if (i->Pin.MulL.srcR->tag == Pri_Imm) {
          vex_printf("mulli ");
       } else {
          vex_printf("mul%s%c ",
@@ -809,18 +809,18 @@ void ppPPC32Instr ( PPC32Instr* i )
       }
       ppHRegPPC32(i->Pin.MulL.dst);
       vex_printf(",");
-      ppHRegPPC32(i->Pin.MulL.src1);
+      ppHRegPPC32(i->Pin.MulL.srcL);
       vex_printf(",");
-      ppPPC32RI(i->Pin.MulL.src2);
+      ppPPC32RI(i->Pin.MulL.srcR);
       return;
    case Pin_Div:
       vex_printf("divw%s ",
                  i->Pin.Div.syned ? "" : "u");
       ppHRegPPC32(i->Pin.Div.dst);
       vex_printf(",");
-      ppHRegPPC32(i->Pin.Div.src1);
+      ppHRegPPC32(i->Pin.Div.srcL);
       vex_printf(",");
-      ppHRegPPC32(i->Pin.Div.src2);
+      ppHRegPPC32(i->Pin.Div.srcR);
       return;
 //..       case Xin_Sh3232:
 //..          vex_printf("%sdl ", showX86ShiftOp(i->Xin.Sh3232.op));
@@ -1089,14 +1089,14 @@ void getRegUsage_PPC32Instr ( HRegUsage* u, PPC32Instr* i )
    initHRegUsage(u);
    switch (i->tag) {
    case Pin_Alu32:
-      addHRegUse(u, HRmRead, i->Pin.Alu32.src1);
-      addRegUsage_PPC32RI(u, i->Pin.Alu32.src2);
+      addHRegUse(u, HRmRead, i->Pin.Alu32.srcL);
+      addRegUsage_PPC32RI(u, i->Pin.Alu32.srcR);
       addHRegUse(u, HRmWrite, i->Pin.Alu32.dst);
       return;
 
    case Pin_Sub32:
-      addRegUsage_PPC32RI(u, i->Pin.Sub32.src1);
-      addHRegUse(u, HRmRead, i->Pin.Sub32.src2);
+      addRegUsage_PPC32RI(u, i->Pin.Sub32.srcL);
+      addHRegUse(u, HRmRead, i->Pin.Sub32.srcR);
       addHRegUse(u, HRmWrite, i->Pin.Sub32.dst);
       return;
 
@@ -1107,8 +1107,8 @@ void getRegUsage_PPC32Instr ( HRegUsage* u, PPC32Instr* i )
       return;
       
    case Pin_Cmp32:
-      addHRegUse(u, HRmRead, i->Pin.Cmp32.src1);
-      addRegUsage_PPC32RI(u, i->Pin.Cmp32.src2);
+      addHRegUse(u, HRmRead, i->Pin.Cmp32.srcL);
+      addRegUsage_PPC32RI(u, i->Pin.Cmp32.srcR);
       return;
 
    case Pin_Unary32:
@@ -1117,13 +1117,13 @@ void getRegUsage_PPC32Instr ( HRegUsage* u, PPC32Instr* i )
       return;
    case Pin_MulL:
       addHRegUse(u, HRmWrite, i->Pin.MulL.dst);
-      addHRegUse(u, HRmRead, i->Pin.MulL.src1);
-      addRegUsage_PPC32RI(u, i->Pin.MulL.src2);
+      addHRegUse(u, HRmRead, i->Pin.MulL.srcL);
+      addRegUsage_PPC32RI(u, i->Pin.MulL.srcR);
       return;
    case Pin_Div:
       addHRegUse(u, HRmWrite, i->Pin.Div.dst);
-      addHRegUse(u, HRmRead, i->Pin.Div.src1);
-      addHRegUse(u, HRmRead, i->Pin.Div.src2);
+      addHRegUse(u, HRmRead, i->Pin.Div.srcL);
+      addHRegUse(u, HRmRead, i->Pin.Div.srcR);
       return;
 //..       case Xin_Sh3232:
 //..          addHRegUse(u, HRmRead, i->Xin.Sh3232.src);
@@ -1333,13 +1333,13 @@ void mapRegs_PPC32Instr (HRegRemap* m, PPC32Instr* i)
    switch (i->tag) {
    case Pin_Alu32:
       mapReg(m, &i->Pin.Alu32.dst);
-      mapReg(m, &i->Pin.Alu32.src1);
-      mapRegs_PPC32RI(m, i->Pin.Alu32.src2);
+      mapReg(m, &i->Pin.Alu32.srcL);
+      mapRegs_PPC32RI(m, i->Pin.Alu32.srcR);
       return;
    case Pin_Sub32:
       mapReg(m, &i->Pin.Sub32.dst);
-      mapRegs_PPC32RI(m, i->Pin.Sub32.src1);
-      mapReg(m, &i->Pin.Sub32.src2);
+      mapRegs_PPC32RI(m, i->Pin.Sub32.srcL);
+      mapReg(m, &i->Pin.Sub32.srcR);
       return;
    case Pin_Sh32:
       mapReg(m, &i->Pin.Sh32.dst);
@@ -1347,8 +1347,8 @@ void mapRegs_PPC32Instr (HRegRemap* m, PPC32Instr* i)
       mapRegs_PPC32RI(m, i->Pin.Sh32.shft);
       return;
    case Pin_Cmp32:
-      mapReg(m, &i->Pin.Cmp32.src1);
-      mapRegs_PPC32RI(m, i->Pin.Cmp32.src2);
+      mapReg(m, &i->Pin.Cmp32.srcL);
+      mapRegs_PPC32RI(m, i->Pin.Cmp32.srcR);
       return;
    case Pin_Unary32:
       mapReg(m, &i->Pin.Unary32.dst);
@@ -1356,13 +1356,13 @@ void mapRegs_PPC32Instr (HRegRemap* m, PPC32Instr* i)
       return;
    case Pin_MulL:
       mapReg(m, &i->Pin.MulL.dst);
-      mapReg(m, &i->Pin.MulL.src1);
-      mapRegs_PPC32RI(m, i->Pin.MulL.src2);
+      mapReg(m, &i->Pin.MulL.srcL);
+      mapRegs_PPC32RI(m, i->Pin.MulL.srcR);
       return;
    case Pin_Div:
       mapReg(m, &i->Pin.Div.dst);
-      mapReg(m, &i->Pin.Div.src1);
-      mapReg(m, &i->Pin.Div.src2);
+      mapReg(m, &i->Pin.Div.srcL);
+      mapReg(m, &i->Pin.Div.srcR);
       return;
 //..       case Xin_Sh3232:
 //..          mapReg(m, &i->Xin.Sh3232.src);
@@ -1493,11 +1493,11 @@ Bool isMove_PPC32Instr ( PPC32Instr* i, HReg* src, HReg* dst )
       // or Rd,Rs,Rs == mr Rd,Rs
       if (i->Pin.Alu32.op != Palu_OR)
          return False;
-      if (i->Pin.Alu32.src2->tag != Pri_Reg)
+      if (i->Pin.Alu32.srcR->tag != Pri_Reg)
          return False;
-      if (i->Pin.Alu32.src2->Pri.Reg.reg != i->Pin.Alu32.src1)
+      if (i->Pin.Alu32.srcR->Pri.Reg.reg != i->Pin.Alu32.srcL)
          return False;
-      *src = i->Pin.Alu32.src1;
+      *src = i->Pin.Alu32.srcL;
       *dst = i->Pin.Alu32.dst;
       return True;
    }
@@ -1808,72 +1808,70 @@ static UChar* emit32 ( UChar* p, UInt w32 )
 //..    return p;
 //.. }
 
-static UChar* mkFormD ( UChar* p, UInt op1, UInt r1, UInt r2, UInt imm )
+static UChar* mkFormD ( UChar* p, UInt opc1, UInt r1, UInt r2, UInt imm )
 {
    UInt theInstr;
-   vassert(op1 < 0x40);
+   vassert(opc1 < 0x40);
    vassert(r1  < 0x20);
    vassert(r2  < 0x20);
    vassert(imm < 0x10000 || imm >= 0xFFFF8000);     // Pos|Neg
    imm = imm & 0xFFFF;
-   theInstr = ((op1<<26) | (r1<<21) | (r2<<16) | (imm));
+   theInstr = ((opc1<<26) | (r1<<21) | (r2<<16) | (imm));
    return emit32(p, theInstr);
 }
 
-static UChar* mkFormX ( UChar* p, UInt op1, UInt r1, UInt r2,
-                        UInt r3, UInt op2, UInt b0 )
+static UChar* mkFormX ( UChar* p, UInt opc1, UInt r1, UInt r2,
+                        UInt r3, UInt opc2, UInt b0 )
 {
    UInt theInstr;
-   vassert(op1 < 0x40);
+   vassert(opc1 < 0x40);
    vassert(r1  < 0x20);
    vassert(r2  < 0x20);
    vassert(r3  < 0x20);
-   vassert(op2 < 0x400);
+   vassert(opc2 < 0x400);
    vassert(b0  < 0x2);
-   theInstr = ((op1<<26) | (r1<<21) | (r2<<16) |
-               (r3<<11) | (op2<<1) | (b0));
+   theInstr = ((opc1<<26) | (r1<<21) | (r2<<16) | (r3<<11) | (opc2<<1) | (b0));
    return emit32(p, theInstr);
 }
 
-static UChar* mkFormXO ( UChar* p, UInt op1, UInt r1, UInt r2,
-                         UInt r3, UInt b10, UInt op2, UInt b0 )
+static UChar* mkFormXO ( UChar* p, UInt opc1, UInt r1, UInt r2,
+                         UInt r3, UInt b10, UInt opc2, UInt b0 )
 {
    UInt theInstr;
-   vassert(op1 < 0x40);
+   vassert(opc1 < 0x40);
    vassert(r1  < 0x20);
    vassert(r2  < 0x20);
    vassert(r3  < 0x20);
    vassert(b10 < 0x2);
-   vassert(op2 < 0x200);
+   vassert(opc2 < 0x200);
    vassert(b0  < 0x2);
-   theInstr = ((op1<<26) | (r1<<21) | (r2<<16) |
-               (r3<<11) | (b10 << 10) | (op2<<1) | (b0));
+   theInstr = ((opc1<<26) | (r1<<21) | (r2<<16) |
+               (r3<<11) | (b10 << 10) | (opc2<<1) | (b0));
    return emit32(p, theInstr);
 }
 
-static UChar* mkFormXL ( UChar* p, UInt op1, UInt f1, UInt f2,
-                         UInt f3, UInt op2, UInt b0 )
+static UChar* mkFormXL ( UChar* p, UInt opc1, UInt f1, UInt f2,
+                         UInt f3, UInt opc2, UInt b0 )
 {
    UInt theInstr;
-   vassert(op1 < 0x40);
+   vassert(opc1 < 0x40);
    vassert(f1  < 0x20);
    vassert(f2  < 0x20);
    vassert(f3  < 0x20);
-   vassert(op2 < 0x400);
+   vassert(opc2 < 0x400);
    vassert(b0  < 0x2);
-   theInstr = ((op1<<26) | (f1<<21) | (f2<<16) |
-               (f3<<11) | (op2<<1) | (b0));
+   theInstr = ((opc1<<26) | (f1<<21) | (f2<<16) | (f3<<11) | (opc2<<1) | (b0));
    return emit32(p, theInstr);
 }
 
 // Note: for split field ops, give mnemonic arg
-static UChar* mkFormXFX ( UChar* p, UInt r1, UInt f2, UInt op2 )
+static UChar* mkFormXFX ( UChar* p, UInt r1, UInt f2, UInt opc2 )
 {
    UInt theInstr;
    vassert(r1  < 0x20);
    vassert(f2  < 0x20);
-   vassert(op2 < 0x400);
-   switch (op2) {
+   vassert(opc2 < 0x400);
+   switch (opc2) {
    case 144:  // mtcrf
       vassert(f2 < 0x100);
       f2 = f2 << 1;
@@ -1886,7 +1884,7 @@ static UChar* mkFormXFX ( UChar* p, UInt r1, UInt f2, UInt op2 )
       break;
    default: vpanic("mkFormXFX(PPC32)");
    }
-   theInstr = ((31<<26) | (r1<<21) | (f2<<11) | (op2<<1));
+   theInstr = ((31<<26) | (r1<<21) | (f2<<11) | (opc2<<1));
    return emit32(p, theInstr);
 }
 
@@ -1913,52 +1911,54 @@ static UChar* mkFormB ( UChar* p, UInt BO, UInt BI,
    vassert(BD  < 0x4000);
    vassert(AA  < 0x2);
    vassert(LK  < 0x2);
-   theInstr = ((16<<26) | (BO<<21) | (BI<<16) |
-               (BD<<2) | (AA<<1) | (LK));
+   theInstr = ((16<<26) | (BO<<21) | (BI<<16) | (BD<<2) | (AA<<1) | (LK));
    return emit32(p, theInstr);
 }
 
 // rotates
-static UChar* mkFormM ( UChar* p, UInt op1, UInt r1, UInt r2,
+static UChar* mkFormM ( UChar* p, UInt opc1, UInt r1, UInt r2,
                         UInt f3, UInt MB, UInt ME, UInt Rc )
 {
    UInt theInstr;
-   vassert(op1 < 0x40);
+   vassert(opc1 < 0x40);
    vassert(r1  < 0x20);
    vassert(r2  < 0x20);
    vassert(f3  < 0x20);
    vassert(MB  < 0x20);
    vassert(ME  < 0x20);
    vassert(Rc  < 0x2);
-   theInstr = ((op1<<26) | (r1<<21) | (r2<<16) |
+   theInstr = ((opc1<<26) | (r1<<21) | (r2<<16) |
                (f3<<11) | (MB<<6) | (ME<<1) | (Rc));
    return emit32(p, theInstr);
 }
 
-static UChar* doAMode_IR ( UChar* p, UInt op1, HReg hrSD, PPC32AMode* am )
+static UChar* doAMode_IR ( UChar* p, UInt opc1, HReg hrSD, PPC32AMode* am )
 {
    UInt rSD, rA, idx;
    vassert(am->tag == Pam_IR);
+   vassert(am->Pam.IR.index < 0x10000);
+
    rSD = iregNo(hrSD);
    rA  = iregNo(am->Pam.IR.base);
    idx = am->Pam.IR.index;
-   vassert(idx < 0x10000);
-   
-   p = mkFormD(p, op1, rSD, rA, idx);
+
+   p = mkFormD(p, opc1, rSD, rA, idx);
    return p;
 }
 
 
-static UChar* doAMode_RR ( UChar* p, UInt op1, UInt op2,
+static UChar* doAMode_RR ( UChar* p, UInt opc1, UInt opc2,
                            HReg hrSD, PPC32AMode* am )
 {
    UInt rSD, rA, rB;
 //   vassert(hregClass(hrSD) == HRcInt32); // CAB: etc. worth doing this?
    vassert(am->tag == Pam_RR);
+
    rSD = iregNo(hrSD);
    rA  = iregNo(am->Pam.RR.base);
    rB  = iregNo(am->Pam.RR.index);
-   p = mkFormX(p, op1, rSD, rA, rB, op2, 0);
+   
+   p = mkFormX(p, opc1, rSD, rA, rB, opc2, 0);
    return p;
 }
 
@@ -2017,64 +2017,64 @@ Int emit_PPC32Instr ( UChar* buf, Int nbuf, PPC32Instr* i )
    switch (i->tag) {
 
    case Pin_Alu32: {
-      UInt op1, op2, rR, immR;
-      UInt rD  = iregNo(i->Pin.Alu32.dst);
-      UInt rL = iregNo(i->Pin.Alu32.src1);
+      UInt opc1, opc2, r_srcR, imm_srcR;
+      UInt r_dst  = iregNo(i->Pin.Alu32.dst);
+      UInt r_srcL = iregNo(i->Pin.Alu32.srcL);
 
       /* ADD/AND/OR/XOR */
-      if (i->Pin.Alu32.src2->tag == Pri_Reg) {
-         op1 = 31;
-         rR = iregNo(i->Pin.Alu32.src2->Pri.Reg.reg);
+      if (i->Pin.Alu32.srcR->tag == Pri_Reg) {
+         opc1 = 31;
+         r_srcR = iregNo(i->Pin.Alu32.srcR->Pri.Reg.reg);
          switch (i->Pin.Alu32.op) {
-         case Palu_ADD: op2 = 266; break;
-         case Palu_AND: op2 =  28; break;
-         case Palu_XOR: op2 = 316; break;
-         case Palu_OR:  op2 = 444; break;
+         case Palu_ADD: opc2 = 266; break;
+         case Palu_AND: opc2 =  28; break;
+         case Palu_XOR: opc2 = 316; break;
+         case Palu_OR:  opc2 = 444; break;
          default:
             goto bad;
          }
 
          if (i->Pin.Alu32.op == Palu_ADD) {
-            p = mkFormXO(p, op1, rD, rL, rR, 0, op2, 0);
+            p = mkFormXO(p, opc1, r_dst, r_srcL, r_srcR, 0, opc2, 0);
          } else {
-            p = mkFormX(p, op1, rL, rD, rR, op2, 0);
+            p = mkFormX(p, opc1, r_srcL, r_dst, r_srcR, opc2, 0);
          }
       } else { // Pri_Imm:
-         immR = i->Pin.Alu32.src2->Pri.Imm.imm32;
+         imm_srcR = i->Pin.Alu32.srcR->Pri.Imm.imm32;
          switch (i->Pin.Alu32.op) {
-         case Palu_ADD: op1 = 14; break;
-         case Palu_AND: op1 = 28; break;
-         case Palu_XOR: op1 = 26; break;
-         case Palu_OR:  op1 = 24; break;
+         case Palu_ADD: opc1 = 14; break;
+         case Palu_AND: opc1 = 28; break;
+         case Palu_XOR: opc1 = 26; break;
+         case Palu_OR:  opc1 = 24; break;
          default:
             goto bad;
          }
 
          if (i->Pin.Alu32.op == Palu_ADD) {
-            p = mkFormD(p, op1, rD, rL, immR);
+            p = mkFormD(p, opc1, r_dst, r_srcL, imm_srcR);
          } else {
-            p = mkFormD(p, op1, rL, rD, immR);
+            p = mkFormD(p, opc1, r_srcL, r_dst, imm_srcR);
          }
       }
       goto done;
    }
 
    case Pin_Sub32: {
-      UInt rD = iregNo(i->Pin.Sub32.dst);
-      UInt rR = iregNo(i->Pin.Sub32.src2);
-      UInt rL, immL;
+      UInt r_dst = iregNo(i->Pin.Sub32.dst);
+      UInt r_srcR = iregNo(i->Pin.Sub32.srcR);
+      UInt r_srcL, imm_srcL;
 
       // Note argument swap: PPC32 only has sub-from instrs
-      switch (i->Pin.Sub32.src1->tag) {
+      switch (i->Pin.Sub32.srcL->tag) {
       case Pri_Reg:
-         rL = iregNo(i->Pin.Sub32.src1->Pri.Reg.reg);
-         // subf rD, rR, rL
-         p = mkFormXO(p, 31, rD, rR, rL, 0, 40, 0);
+         r_srcL = iregNo(i->Pin.Sub32.srcL->Pri.Reg.reg);
+         // subf r_dst, r_srcR, r_srcL
+         p = mkFormXO(p, 31, r_dst, r_srcR, r_srcL, 0, 40, 0);
          break;
       case Pri_Imm:
-         immL = i->Pin.Sub32.src1->Pri.Imm.imm32;
-         // subf rD, rR, immL
-         p = mkFormD(p, 8, rD, rR, immL);
+         imm_srcL = i->Pin.Sub32.srcL->Pri.Imm.imm32;
+         // subf r_dst, r_srcR, imm_srcL
+         p = mkFormD(p, 8, r_dst, r_srcR, imm_srcL);
          break;
       default:
          goto bad;
@@ -2083,10 +2083,10 @@ Int emit_PPC32Instr ( UChar* buf, Int nbuf, PPC32Instr* i )
    }
 
    case Pin_Sh32: {
-      UInt opc1 = 31, opc2, rB, sh;
+      UInt opc1 = 31, opc2, r_shft, imm_shft;
       UInt op = i->Pin.Sh32.op;
-      UInt rS = iregNo(i->Pin.Sh32.src);
-      UInt rA = iregNo(i->Pin.Sh32.dst);
+      UInt r_src = iregNo(i->Pin.Sh32.src);
+      UInt r_dst = iregNo(i->Pin.Sh32.dst);
 
       switch (i->Pin.Sh32.shft->tag) {
       case Pri_Reg:
@@ -2096,15 +2096,15 @@ Int emit_PPC32Instr ( UChar* buf, Int nbuf, PPC32Instr* i )
          case Psh_SAR: opc2 = 792; break;
          default: goto bad;
          }
-         rB = iregNo(i->Pin.Sh32.shft->Pri.Reg.reg);
-         p = mkFormX(p, opc1, rS, rA, rB, opc2, 0);
+         r_shft = iregNo(i->Pin.Sh32.shft->Pri.Reg.reg);
+         p = mkFormX(p, opc1, r_src, r_dst, r_shft, opc2, 0);
          break;
 
       case Pri_Imm:  // SAR only
          if (op != Psh_SAR) goto bad;
          opc2 = 824;
-         sh = i->Pin.Sh32.shft->Pri.Imm.imm32;
-         p = mkFormX(p, opc1, rS, rA, sh, opc2, 0);
+         imm_shft = i->Pin.Sh32.shft->Pri.Imm.imm32;
+         p = mkFormX(p, opc1, r_src, r_dst, imm_shft, opc2, 0);
          break;
 
       default:
@@ -2117,20 +2117,20 @@ Int emit_PPC32Instr ( UChar* buf, Int nbuf, PPC32Instr* i )
       UInt opc1, opc2=0;
       UInt op = i->Pin.Cmp32.op;
       UInt fld1 = (7 - i->Pin.Cmp32.crfD) << 2;
-      UInt r1 = iregNo(i->Pin.Cmp32.src1);
-      UInt r2, imm;
+      UInt r_srcL = iregNo(i->Pin.Cmp32.srcL);
+      UInt r_srcR, imm_srcR;
 
-      switch (i->Pin.Cmp32.src2->tag) {
+      switch (i->Pin.Cmp32.srcR->tag) {
       case Pri_Imm:
          opc1 = (op == Pcmp_U) ? 10 : 11;
-         imm = i->Pin.Cmp32.src2->Pri.Imm.imm32;
-         p = mkFormD(p, opc1, fld1, r1, imm);
+         imm_srcR = i->Pin.Cmp32.srcR->Pri.Imm.imm32;
+         p = mkFormD(p, opc1, fld1, r_srcL, imm_srcR);
          break;
       case Pri_Reg:
          opc1 = 31;
          opc2 = (op == Pcmp_U) ? 32 : 0;
-         r2 = iregNo(i->Pin.Cmp32.src2->Pri.Reg.reg);
-         p = mkFormX(p, opc1, fld1, r1, r2, opc2, 0);
+         r_srcR = iregNo(i->Pin.Cmp32.srcR->Pri.Reg.reg);
+         p = mkFormX(p, opc1, fld1, r_srcL, r_srcR, opc2, 0);
          break;
       default: goto bad;
       }        
@@ -2159,23 +2159,23 @@ Int emit_PPC32Instr ( UChar* buf, Int nbuf, PPC32Instr* i )
    case Pin_MulL: {
       Bool syned  = i->Pin.MulL.syned;
       UInt r_dst  = iregNo(i->Pin.MulL.dst);
-      UInt r_src1 = iregNo(i->Pin.MulL.src1);
-      UInt r_src2, imm;
+      UInt r_srcL = iregNo(i->Pin.MulL.srcL);
+      UInt r_srcR, imm;
 
       switch (i->Pin.MulL.word) {
       case 0:  // Mult LOW word
-         switch (i->Pin.MulL.src2->tag) {
+         switch (i->Pin.MulL.srcR->tag) {
          case Pri_Reg:
-            // mullw r_dst,r_src1,r_src2
+            // mullw r_dst,r_srcL,r_srcR
             // sign makes no difference.
-            r_src2 = iregNo(i->Pin.MulL.src2->Pri.Reg.reg);
-            p = mkFormXO(p, 31, r_dst, r_src1, r_src2, 0, 235, 0);
+            r_srcR = iregNo(i->Pin.MulL.srcR->Pri.Reg.reg);
+            p = mkFormXO(p, 31, r_dst, r_srcL, r_srcR, 0, 235, 0);
             break;
          case Pri_Imm:
             // mulli r_dst,r_src,imm
             vassert(syned == True);  // always signed
-            imm = i->Pin.MulL.src2->Pri.Imm.imm32;
-            p = mkFormD(p, 07, r_dst, r_src1, imm);
+            imm = i->Pin.MulL.srcR->Pri.Imm.imm32;
+            p = mkFormD(p, 07, r_dst, r_srcL, imm);
             break;
          default:
             goto bad;
@@ -2183,14 +2183,14 @@ Int emit_PPC32Instr ( UChar* buf, Int nbuf, PPC32Instr* i )
          break;
 
       case 1:  // Mult HIGH word
-         vassert(i->Pin.MulL.src2->tag == Pri_Reg);
-         r_src2 = iregNo(i->Pin.MulL.src2->Pri.Reg.reg);
+         vassert(i->Pin.MulL.srcR->tag == Pri_Reg);
+         r_srcR = iregNo(i->Pin.MulL.srcR->Pri.Reg.reg);
          if (syned == True) {
-            // mulhw r_dst,r_src1,r_src2
-            p = mkFormXO(p, 31, r_dst, r_src1, r_src2, 0, 75, 0);
+            // mulhw r_dst,r_srcL,r_srcR
+            p = mkFormXO(p, 31, r_dst, r_srcL, r_srcR, 0, 75, 0);
          } else {
-            // mulhwu r_dst,r_src1,r_src2
-            p = mkFormXO(p, 31, r_dst, r_src1, r_src2, 0, 11, 0);
+            // mulhwu r_dst,r_srcL,r_srcR
+            p = mkFormXO(p, 31, r_dst, r_srcL, r_srcR, 0, 11, 0);
          }
          break;
 
@@ -2202,15 +2202,15 @@ Int emit_PPC32Instr ( UChar* buf, Int nbuf, PPC32Instr* i )
    case Pin_Div: {
       Bool syned  = i->Pin.MulL.syned;
       UInt r_dst  = iregNo(i->Pin.Div.dst);
-      UInt r_src1 = iregNo(i->Pin.Div.src1);
-      UInt r_src2 = iregNo(i->Pin.Div.src2);
+      UInt r_srcL = iregNo(i->Pin.Div.srcL);
+      UInt r_srcR = iregNo(i->Pin.Div.srcR);
 
       if (syned == True) {
-         // divw r_dst,r_src1,r_src2
-         p = mkFormXO(p, 31, r_dst, r_src1, r_src2, 0, 491, 0);
+         // divw r_dst,r_srcL,r_srcR
+         p = mkFormXO(p, 31, r_dst, r_srcL, r_srcR, 0, 491, 0);
       } else {
-         // divwu r_dst,r_src1,r_src2
-         p = mkFormXO(p, 31, r_dst, r_src1, r_src2, 0, 459, 0);
+         // divwu r_dst,r_srcL,r_srcR
+         p = mkFormXO(p, 31, r_dst, r_srcL, r_srcR, 0, 459, 0);
       }
       goto done;
    }
@@ -2285,7 +2285,7 @@ Int emit_PPC32Instr ( UChar* buf, Int nbuf, PPC32Instr* i )
       UInt magic_num = 0;
       UChar r_return = 3;    /* Put target addr into %r3 */
       PPC32CondCode cond = i->Pin.Goto.cond;
-      UInt imm;
+      UInt r_dst, imm_dst;
       
       /* First off, if this is conditional, create a conditional
          jump over the rest of it. */
@@ -2323,10 +2323,9 @@ Int emit_PPC32Instr ( UChar* buf, Int nbuf, PPC32Instr* i )
 
       /* Get the destination address into %r_return */
       if (i->Pin.Goto.dst->tag == Pri_Imm) {
-         imm = i->Pin.Goto.dst->Pri.Imm.imm32;
-         p = mkLoadImm(p, r_return, imm);
+         imm_dst = i->Pin.Goto.dst->Pri.Imm.imm32;
+         p = mkLoadImm(p, r_return, imm_dst);
       } else {
-         UInt r_dst;
          vassert(i->Pin.Goto.dst->tag == Pri_Reg);
          r_dst = iregNo(i->Pin.Goto.dst->Pri.Reg.reg);
          p = mkMoveReg(p, r_return, r_dst);
@@ -2346,9 +2345,10 @@ Int emit_PPC32Instr ( UChar* buf, Int nbuf, PPC32Instr* i )
    }
 
    case Pin_CMov32: {
-      UInt r_dst, imm, r_src;
+      UInt r_dst, imm_src, r_src;
       PPC32CondCode cond;
       vassert(i->Pin.CMov32.cond.test != Pct_ALWAYS);
+
       r_dst = iregNo(i->Pin.CMov32.dst);
       cond = i->Pin.CMov32.cond;
 
@@ -2363,8 +2363,8 @@ Int emit_PPC32Instr ( UChar* buf, Int nbuf, PPC32Instr* i )
       // cond true: move src => dst
       switch (i->Pin.CMov32.src->tag) {
       case Pri_Imm:
-         imm = i->Pin.CMov32.src->Pri.Imm.imm32;
-         p = mkLoadImm(p, r_dst, imm);
+         imm_src = i->Pin.CMov32.src->Pri.Imm.imm32;
+         p = mkLoadImm(p, r_dst, imm_src);
          break;
       case Pri_Reg:
          r_src = iregNo(i->Pin.CMov32.src->Pri.Reg.reg);
@@ -2385,26 +2385,26 @@ Int emit_PPC32Instr ( UChar* buf, Int nbuf, PPC32Instr* i )
 
    case Pin_Load: {
       Bool syned = i->Pin.Load.syned;
-      UInt op1, op2, sz = i->Pin.Load.sz;
+      UInt opc1, opc2, sz = i->Pin.Load.sz;
       switch (i->Pin.Load.src->tag) {
       case Pam_IR:
          if (sz == 2) {  // the only signed load
-            op1 = (syned) ? 42: 40;
+            opc1 = (syned) ? 42: 40;
          } else {
             vassert(syned == False);
-            op1 = (sz == 1) ? 34 : 32;   // 1:4
+            opc1 = (sz == 1) ? 34 : 32;   // 1:4
          }
-         p = doAMode_IR(p, op1, i->Pin.Load.dst, i->Pin.Load.src);
+         p = doAMode_IR(p, opc1, i->Pin.Load.dst, i->Pin.Load.src);
          goto done;
       case Pam_RR:
-         op1 = 31;
+         opc1 = 31;
          if (sz == 2) {  // the only signed load
-            op2 = (syned) ? 343: 279;
+            opc2 = (syned) ? 343: 279;
          } else {
             vassert(syned == False);
-            op2 = (sz == 1) ? 87 : 23;   // 1:4
+            opc2 = (sz == 1) ? 87 : 23;   // 1:4
          }
-         p = doAMode_RR(p, op1, op2, i->Pin.Load.dst, i->Pin.Load.src);
+         p = doAMode_RR(p, opc1, opc2, i->Pin.Load.dst, i->Pin.Load.src);
          goto done;
       default:
          goto bad;
@@ -2434,7 +2434,7 @@ Int emit_PPC32Instr ( UChar* buf, Int nbuf, PPC32Instr* i )
          p = mkFormM(p, 21, r_tmp, r_dst, rot_imm, 31, 31, 0);
 
          if (cond.test == Pct_FALSE) {
-            // flip bit  => xori rD,rD,1
+            // flip bit  => xori r_dst,r_dst,1
             p = mkFormD(p, 26, r_dst, r_dst, 1);
          }
       }
@@ -2462,16 +2462,16 @@ Int emit_PPC32Instr ( UChar* buf, Int nbuf, PPC32Instr* i )
    }
 
    case Pin_Store: {
-      UInt op1, op2, sz = i->Pin.Store.sz;
+      UInt opc1, opc2, sz = i->Pin.Store.sz;
       switch (i->Pin.Store.dst->tag) {
       case Pam_IR:
-         op1 = (sz == 1) ? 38 : ((sz == 2) ? 44 : 36);      // 1:2:4
-         p = doAMode_IR(p, op1, i->Pin.Store.src, i->Pin.Store.dst);
+         opc1 = (sz == 1) ? 38 : ((sz == 2) ? 44 : 36);      // 1:2:4
+         p = doAMode_IR(p, opc1, i->Pin.Store.src, i->Pin.Store.dst);
          goto done;
       case Pam_RR:
-         op1 = 31;
-         op2 = (sz == 1) ? 215 : ((sz == 2) ? 407 : 151);   // 1:2:4
-         p = doAMode_RR(p, op1, op2, i->Pin.Store.src, i->Pin.Store.dst);
+         opc1 = 31;
+         opc2 = (sz == 1) ? 215 : ((sz == 2) ? 407 : 151);   // 1:2:4
+         p = doAMode_RR(p, opc1, opc2, i->Pin.Store.src, i->Pin.Store.dst);
          goto done;
       default:
          goto bad;
index 60c088fd3d74f0c7f46dedcb70c081c08a0bc19c..41e4944fd2b68e36226d5ccdb32d159150b7f3c3 100644 (file)
@@ -338,13 +338,13 @@ typedef
          struct {
             PPC32AluOp op;
             HReg       dst;
-            HReg       src1;
-            PPC32RI*   src2;
+            HReg       srcL;
+            PPC32RI*   srcR;
          } Alu32;
          struct {
             HReg       dst;    // PPC32 sub args are switched:
-            PPC32RI*   src1;   // argL => RI
-            HReg       src2;   // argR => R
+            PPC32RI*   srcL;   // argL => RI
+            HReg       srcR;   // argR => R
          } Sub32;
          struct {
             PPC32ShiftOp op;
@@ -355,8 +355,8 @@ typedef
          struct {
             PPC32CmpOp op;
             UInt     crfD;
-            HReg     src1;
-            PPC32RI* src2;
+            HReg     srcL;
+            PPC32RI* srcR;
          } Cmp32;
          /* Not and Neg */
          struct {
@@ -368,15 +368,15 @@ typedef
             Bool     syned;
             Bool     word;   /* low=0, high=1 */
             HReg     dst;
-            HReg     src1;
-            PPC32RI* src2;
+            HReg     srcL;
+            PPC32RI* srcR;
          } MulL;
          /* ppc32 div/divu instruction. */
          struct {
             Bool syned;
             HReg dst;
-            HReg src1;
-            HReg src2;
+            HReg srcL;
+            HReg srcR;
          } Div;
 //..          /* shld/shrd.  op may only be Xsh_SHL or Xsh_SHR */
 //..          struct {
@@ -517,7 +517,7 @@ extern PPC32Instr* PPC32Instr_Sh32      ( PPC32ShiftOp, HReg, HReg, PPC32RI* );
 extern PPC32Instr* PPC32Instr_Cmp32     ( PPC32CmpOp, UInt, HReg, PPC32RI* );
 extern PPC32Instr* PPC32Instr_Unary32   ( PPC32UnaryOp op, HReg dst, HReg src );
 extern PPC32Instr* PPC32Instr_MulL      ( Bool syned, Bool word, HReg, HReg, PPC32RI* );
-extern PPC32Instr* PPC32Instr_Div       ( Bool syned, HReg dst, HReg src1, HReg src2 );
+extern PPC32Instr* PPC32Instr_Div       ( Bool syned, HReg dst, HReg srcL, HReg srcR );
 //.. extern X86Instr* X86Instr_Sh3232    ( X86ShiftOp, UInt amt, HReg src, HReg dst );
 //.. extern X86Instr* X86Instr_Push      ( X86RMI* );
 extern PPC32Instr* PPC32Instr_Call      ( PPC32CondCode, Addr32, Int );
index 194164f3cfd9d574e07b608f602548c8f5a60eab..a9bbd7c666cda16467d2bf06f2e5ca95ef8ecce7 100644 (file)
@@ -286,11 +286,11 @@ static PPC32CondCode iselCondCode     ( ISelEnv* env, IRExpr* e );
 /* Shift helper */
 
 static PPC32Instr* mk_sh32 ( ISelEnv* env, PPC32ShiftOp shOp,
-                             HReg dst, HReg src1, PPC32RI* src2 )
+                             HReg r_dst, HReg r_srcL, PPC32RI* ri_srcR )
 {
    HReg zero, tmp;
-   vassert(hregClass(dst) == HRcInt32);
-   vassert(hregClass(src1) == HRcInt32);
+   vassert(hregClass(r_dst) == HRcInt32);
+   vassert(hregClass(r_srcL) == HRcInt32);
    
    // Note: In this context, GPR0 is NOT read -> just gives _value_ 0
    zero = hregPPC32_GPR0();
@@ -299,19 +299,19 @@ static PPC32Instr* mk_sh32 ( ISelEnv* env, PPC32ShiftOp shOp,
    switch (shOp) {
    case Psh_SHL:
    case Psh_SHR:  // These ops only take regs as args...
-      if (src2->tag == Pri_Imm) {
-         if (src2->Pri.Imm.imm32 < 32) {
-            addInstr(env, PPC32Instr_Alu32(Palu_ADD, tmp, zero, src2));
-            return PPC32Instr_Sh32(shOp, dst, src1, PPC32RI_Reg(tmp));
+      if (ri_srcR->tag == Pri_Imm) {
+         if (ri_srcR->Pri.Imm.imm32 < 32) {
+            addInstr(env, PPC32Instr_Alu32(Palu_ADD, tmp, zero, ri_srcR));
+            return PPC32Instr_Sh32(shOp, r_dst, r_srcL, PPC32RI_Reg(tmp));
          } else {  // shift > 31 => zero
-            return PPC32Instr_Alu32(Palu_ADD, dst, zero, PPC32RI_Imm(0));
+            return PPC32Instr_Alu32(Palu_ADD, r_dst, zero, PPC32RI_Imm(0));
          }
       } else {
-         return PPC32Instr_Sh32(shOp, dst, src1, src2);
+         return PPC32Instr_Sh32(shOp, r_dst, r_srcL, ri_srcR);
       }
 
    case Psh_SAR:  // No problem here.
-      return PPC32Instr_Sh32(Psh_SAR, dst, dst, src2);
+      return PPC32Instr_Sh32(Psh_SAR, r_dst, r_dst, ri_srcR);
 
    default:
       vpanic("");
@@ -320,38 +320,39 @@ static PPC32Instr* mk_sh32 ( ISelEnv* env, PPC32ShiftOp shOp,
 
 /* Make an int reg-reg move. */
 
-static PPC32Instr* mk_iMOVds_RR ( HReg dst, HReg src )
+static PPC32Instr* mk_iMOVds_RR ( HReg r_dst, HReg r_src )
 {
-   vassert(hregClass(dst) == HRcInt32);
-   vassert(hregClass(src) == HRcInt32);
-   return PPC32Instr_Alu32(Palu_OR, dst, src, PPC32RI_Reg(src));
+   vassert(hregClass(r_dst) == HRcInt32);
+   vassert(hregClass(r_src) == HRcInt32);
+   return PPC32Instr_Alu32(Palu_OR, r_dst, r_src, PPC32RI_Reg(r_src));
 }
 
 /* Load an RI to a reg */
 
-static PPC32Instr* mk_iMOVds_RRI ( ISelEnv* env, HReg dst, PPC32RI* src )
+static PPC32Instr* mk_iMOVds_RRI ( ISelEnv* env, HReg r_dst, PPC32RI* ri_src )
 {
    HReg zero;
-   vassert(hregClass(dst) == HRcInt32);
+   vassert(hregClass(r_dst) == HRcInt32);
    // Note: In this context, GPR0 is NOT read -> just gives _value_ 0
    zero = hregPPC32_GPR0();
 
-   if (src->tag == Pri_Imm) {
-      UInt imm = src->Pri.Imm.imm32;
+   if (ri_src->tag == Pri_Imm) {
+      UInt imm = ri_src->Pri.Imm.imm32;
       if (imm >= 0xFFFF8000 || imm <= 0x7FFF) { // sign-extendable from 16 bits?
-         return PPC32Instr_Alu32(Palu_ADD, dst, zero, PPC32RI_Imm(imm & 0xFFFF));
+         return PPC32Instr_Alu32(Palu_ADD, r_dst, zero, PPC32RI_Imm(imm & 0xFFFF));
       }
       if (imm > 0xFFFF) {
          // CAB: addis (aka lis) would be good...
-         addInstr(env, PPC32Instr_Alu32(Palu_ADD, dst, zero, PPC32RI_Imm(imm>>16)));
-         addInstr(env, mk_sh32(env, Psh_SHL, dst, dst, PPC32RI_Imm(16)));
-         return PPC32Instr_Alu32(Palu_OR, dst, dst, PPC32RI_Imm(imm & 0xFFFF));
+         addInstr(env, PPC32Instr_Alu32(Palu_ADD, r_dst, zero, PPC32RI_Imm(imm>>16)));
+         addInstr(env, mk_sh32(env, Psh_SHL, r_dst, r_dst, PPC32RI_Imm(16)));
+         return PPC32Instr_Alu32(Palu_OR, r_dst, r_dst, PPC32RI_Imm(imm & 0xFFFF));
       }
       // Load immediate _without_ sign extend
-      addInstr(env, PPC32Instr_Alu32(Palu_ADD, dst, zero, PPC32RI_Imm(0)));
-      return PPC32Instr_Alu32(Palu_OR, dst, dst, src);
+      addInstr(env, PPC32Instr_Alu32(Palu_ADD, r_dst, zero, PPC32RI_Imm(0)));
+      return PPC32Instr_Alu32(Palu_OR, r_dst, r_dst, ri_src);
    } else {
-      return PPC32Instr_Alu32(Palu_OR, dst, src->Pri.Reg.reg, src);  // mr rD,rS
+      // mr rD,rS
+      return PPC32Instr_Alu32(Palu_OR, r_dst, ri_src->Pri.Reg.reg, ri_src);
    }
 }
 
@@ -833,11 +834,11 @@ static HReg iselIntExpr_R_wrk ( ISelEnv* env, IRExpr* e )
 
    /* --------- LOAD --------- */
    case Iex_LDle: {
-      HReg dst = newVRegI(env);
+      HReg r_dst = newVRegI(env);
       PPC32AMode* am_src = iselIntExpr_AMode(env, e->Iex.LDle.addr);
       if (ty == Ity_I8 || ty == Ity_I16 || ty == Ity_I32) {
-         addInstr(env, PPC32Instr_Load( sizeofIRType(ty), False, dst, am_src ));
-         return dst;
+         addInstr(env, PPC32Instr_Load( sizeofIRType(ty), False, r_dst, am_src ));
+         return r_dst;
       }
       break;
    }
@@ -872,15 +873,17 @@ static HReg iselIntExpr_R_wrk ( ISelEnv* env, IRExpr* e )
       /* For commutative ops we assume any literal
          values are on the second operand. */
       if (aluOp != Palu_INVALID) {
-         HReg dst    = newVRegI(env);
-         HReg src    = iselIntExpr_R(env, e->Iex.Binop.arg1);
+         HReg r_dst  = newVRegI(env);
+         HReg r_srcL = iselIntExpr_R(env, e->Iex.Binop.arg1);
          PPC32RI* ri = iselIntExpr_RI(env, e->Iex.Binop.arg2);
+         PPC32RI* ri_srcR;
          if (aluOp == Palu_ADD) {
-            addInstr(env, PPC32Instr_Alu32(aluOp, dst, src, mk_FitRI16_S(env, ri)));
+            ri_srcR = mk_FitRI16_S(env, ri );
          } else {
-            addInstr(env, PPC32Instr_Alu32(aluOp, dst, src, mk_FitRI16_U(env, ri)));
+            ri_srcR = mk_FitRI16_U(env, ri);
          }
-         return dst;
+         addInstr(env, PPC32Instr_Alu32(aluOp, r_dst, r_srcL, ri_srcR));
+         return r_dst;
       }
 //..       /* Could do better here; forcing the first arg into a reg
 //..          isn't always clever.
@@ -904,38 +907,38 @@ static HReg iselIntExpr_R_wrk ( ISelEnv* env, IRExpr* e )
       if (e->Iex.Binop.op == Iop_Sub8  ||
           e->Iex.Binop.op == Iop_Sub16 ||
           e->Iex.Binop.op == Iop_Sub32) {
-         HReg dst     = newVRegI(env);
+         HReg r_dst     = newVRegI(env);
          PPC32RI* riL = mk_FitRI16_S(env, iselIntExpr_RI(env, e->Iex.Binop.arg1));
          HReg     rR  = iselIntExpr_R(env, e->Iex.Binop.arg2);
-         addInstr(env, PPC32Instr_Sub32(dst, riL, rR));
-         return dst;
+         addInstr(env, PPC32Instr_Sub32(r_dst, riL, rR));
+         return r_dst;
       }
 
       /* How about a div? */
       if (e->Iex.Binop.op == Iop_DivU32) {
-         HReg dst  = newVRegI(env);
-         HReg src1 = iselIntExpr_R(env, e->Iex.Binop.arg1);
-         HReg src2 = iselIntExpr_R(env, e->Iex.Binop.arg2);
-         addInstr(env, PPC32Instr_Div(False, dst, src1, src2));
-         return dst;
+         HReg r_dst  = newVRegI(env);
+         HReg r_srcL = iselIntExpr_R(env, e->Iex.Binop.arg1);
+         HReg r_srcR = iselIntExpr_R(env, e->Iex.Binop.arg2);
+         addInstr(env, PPC32Instr_Div(False, r_dst, r_srcL, r_srcR));
+         return r_dst;
       }
       if (e->Iex.Binop.op == Iop_DivS32) {
-         HReg dst  = newVRegI(env);
-         HReg src1 = iselIntExpr_R(env, e->Iex.Binop.arg1);
-         HReg src2 = iselIntExpr_R(env, e->Iex.Binop.arg2);
-         addInstr(env, PPC32Instr_Div(True, dst, src1, src2));
-         return dst;
+         HReg r_dst  = newVRegI(env);
+         HReg r_srcL = iselIntExpr_R(env, e->Iex.Binop.arg1);
+         HReg r_srcR = iselIntExpr_R(env, e->Iex.Binop.arg2);
+         addInstr(env, PPC32Instr_Div(True, r_dst, r_srcL, r_srcR));
+         return r_dst;
       }
 
       /* No? Anyone for a mul? */
       if (e->Iex.Binop.op == Iop_Mul16 ||
           e->Iex.Binop.op == Iop_Mul32) {
-         Bool syned    = True;
-         HReg dst      = newVRegI(env);
-         HReg src1     = iselIntExpr_R(env, e->Iex.Binop.arg1);
-         PPC32RI* src2 = mk_FitRI16_S(env, iselIntExpr_RI(env, e->Iex.Binop.arg2));
-         addInstr(env, PPC32Instr_MulL(syned, 0, dst, src1, src2));
-         return dst;
+         Bool syned       = True;
+         HReg r_dst         = newVRegI(env);
+         HReg r_srcL      = iselIntExpr_R(env, e->Iex.Binop.arg1);
+         PPC32RI* ri_srcR = mk_FitRI16_S(env, iselIntExpr_RI(env, e->Iex.Binop.arg2));
+         addInstr(env, PPC32Instr_MulL(syned, 0, r_dst, r_srcL, ri_srcR));
+         return r_dst;
       }      
 
       /* Perhaps a shift op? */
@@ -950,26 +953,26 @@ static HReg iselIntExpr_R_wrk ( ISelEnv* env, IRExpr* e )
          shOp = Psh_INVALID; break;
       }
       if (shOp != Psh_INVALID) {
-         HReg dst = newVRegI(env);
+         HReg r_dst = newVRegI(env);
 
          /* regL = the value to be shifted */
-         HReg src = iselIntExpr_R(env, e->Iex.Binop.arg1);
+         HReg r_src = iselIntExpr_R(env, e->Iex.Binop.arg1);
 
          /* Do any necessary widening for 16/8 bit operands */
          switch (e->Iex.Binop.op) {
          case Iop_Shr8:
-            addInstr(env, PPC32Instr_Alu32(Palu_AND, dst, dst, PPC32RI_Imm(0xFF)));
+            addInstr(env, PPC32Instr_Alu32(Palu_AND, r_dst, r_dst, PPC32RI_Imm(0xFF)));
             break;
          case Iop_Shr16:
-            addInstr(env, PPC32Instr_Alu32(Palu_AND, dst, dst, PPC32RI_Imm(0xFFFF)));
+            addInstr(env, PPC32Instr_Alu32(Palu_AND, r_dst, r_dst, PPC32RI_Imm(0xFFFF)));
             break;
          case Iop_Sar8:
-            addInstr(env, mk_sh32(env, Psh_SHL, dst, dst, PPC32RI_Imm(24)));
-            addInstr(env, mk_sh32(env, Psh_SAR, dst, dst, PPC32RI_Imm(24)));
+            addInstr(env, mk_sh32(env, Psh_SHL, r_dst, r_dst, PPC32RI_Imm(24)));
+            addInstr(env, mk_sh32(env, Psh_SAR, r_dst, r_dst, PPC32RI_Imm(24)));
             break;
          case Iop_Sar16:
-            addInstr(env, mk_sh32(env, Psh_SHL, dst, dst, PPC32RI_Imm(16)));
-            addInstr(env, mk_sh32(env, Psh_SAR, dst, dst, PPC32RI_Imm(16)));
+            addInstr(env, mk_sh32(env, Psh_SHL, r_dst, r_dst, PPC32RI_Imm(16)));
+            addInstr(env, mk_sh32(env, Psh_SAR, r_dst, r_dst, PPC32RI_Imm(16)));
             break;
          default: break;
          }
@@ -979,18 +982,18 @@ static HReg iselIntExpr_R_wrk ( ISelEnv* env, IRExpr* e )
          if (e->Iex.Binop.arg2->tag == Iex_Const &&
              e->Iex.Binop.arg2->Iex.Const.con->Ico.U8 < 32) {
             /* assert that the IR is well-typed */
-            Int nshift;
+            Int imm_shft;
             vassert(e->Iex.Binop.arg2->Iex.Const.con->tag == Ico_U8);
-            nshift = e->Iex.Binop.arg2->Iex.Const.con->Ico.U8;
-            vassert(nshift >= 0);
-            if (nshift > 0)
-               addInstr(env, mk_sh32(env, shOp, dst, src, PPC32RI_Imm(nshift)));
+            imm_shft = e->Iex.Binop.arg2->Iex.Const.con->Ico.U8;
+            vassert(imm_shft >= 0);
+            if (imm_shft > 0)
+               addInstr(env, mk_sh32(env, shOp, r_dst, r_src, PPC32RI_Imm(imm_shft)));
          } else {
             /* General case; we have to force the amount into %cl. */
-            HReg shft = iselIntExpr_R(env, e->Iex.Binop.arg2);
-            addInstr(env, mk_sh32(env, shOp, dst, src, PPC32RI_Reg(shft)));
+            HReg r_shft = iselIntExpr_R(env, e->Iex.Binop.arg2);
+            addInstr(env, mk_sh32(env, shOp, r_dst, r_src, PPC32RI_Reg(r_shft)));
          }
-         return dst;
+         return r_dst;
       }
 
       /* Handle misc other ops. */
@@ -1122,10 +1125,10 @@ static HReg iselIntExpr_R_wrk ( ISelEnv* env, IRExpr* e )
                      unop(Iop_1Uto8,unop(Iop_32to1,bind(0))));
       if (matchIRExpr(&mi,p_32to1_then_1Uto8,e)) {
          IRExpr* expr32 = mi.bindee[0];
-         HReg dst = newVRegI(env);
-         HReg src = iselIntExpr_R(env, expr32);
-         addInstr(env, PPC32Instr_Alu32(Palu_AND, dst, src, PPC32RI_Imm(1)));
-         return dst;
+         HReg r_dst = newVRegI(env);
+         HReg r_src = iselIntExpr_R(env, expr32);
+         addInstr(env, PPC32Instr_Alu32(Palu_AND, r_dst, r_src, PPC32RI_Imm(1)));
+         return r_dst;
       }
 
       /* 16Uto32(LDle(expr32)) */
@@ -1134,10 +1137,10 @@ static HReg iselIntExpr_R_wrk ( ISelEnv* env, IRExpr* e )
          DEFINE_PATTERN(p_LDle16_then_16Uto32,
             unop(Iop_16Uto32,IRExpr_LDle(Ity_I16,bind(0))) );
          if (matchIRExpr(&mi,p_LDle16_then_16Uto32,e)) {
-            HReg dst = newVRegI(env);
+            HReg r_dst = newVRegI(env);
             PPC32AMode* amode = iselIntExpr_AMode ( env, mi.bindee[0] );
-            addInstr(env, PPC32Instr_Load(2,False,dst,amode));
-            return dst;
+            addInstr(env, PPC32Instr_Load(2,False,r_dst,amode));
+            return r_dst;
          }
       }
 
@@ -1145,29 +1148,29 @@ static HReg iselIntExpr_R_wrk ( ISelEnv* env, IRExpr* e )
       case Iop_8Uto16:
       case Iop_8Uto32:
       case Iop_16Uto32: {
-         HReg dst = newVRegI(env);
-         HReg src = iselIntExpr_R(env, e->Iex.Unop.arg);
-         UInt mask = e->Iex.Unop.op==Iop_16Uto32 ? 0xFFFF : 0xFF;
-         addInstr(env, PPC32Instr_Alu32(Palu_AND,dst,src,PPC32RI_Imm(mask)));
-         return dst;
+         HReg r_dst = newVRegI(env);
+         HReg r_src = iselIntExpr_R(env, e->Iex.Unop.arg);
+         UInt mask  = e->Iex.Unop.op==Iop_16Uto32 ? 0xFFFF : 0xFF;
+         addInstr(env, PPC32Instr_Alu32(Palu_AND,r_dst,r_src,PPC32RI_Imm(mask)));
+         return r_dst;
       }
       case Iop_8Sto16:
       case Iop_8Sto32:
       case Iop_16Sto32: {
-         HReg dst = newVRegI(env);
-         HReg src = iselIntExpr_R(env, e->Iex.Unop.arg);
-         UInt amt = e->Iex.Unop.op==Iop_16Sto32 ? 16 : 24;
-         addInstr(env, mk_sh32(env, Psh_SHL, dst, src, PPC32RI_Imm(amt)));
-         addInstr(env, mk_sh32(env, Psh_SAR, dst, dst, PPC32RI_Imm(amt)));
-         return dst;
+         HReg r_dst = newVRegI(env);
+         HReg r_src = iselIntExpr_R(env, e->Iex.Unop.arg);
+         UInt amt   = e->Iex.Unop.op==Iop_16Sto32 ? 16 : 24;
+         addInstr(env, mk_sh32(env, Psh_SHL, r_dst, r_src, PPC32RI_Imm(amt)));
+         addInstr(env, mk_sh32(env, Psh_SAR, r_dst, r_dst, PPC32RI_Imm(amt)));
+         return r_dst;
       }
       case Iop_Not8:
       case Iop_Not16:
       case Iop_Not32: {
-         HReg dst = newVRegI(env);
-         HReg src = iselIntExpr_R(env, e->Iex.Unop.arg);
-         addInstr(env, PPC32Instr_Unary32(Pun_NOT,dst,src));
-         return dst;
+         HReg r_dst = newVRegI(env);
+         HReg r_src = iselIntExpr_R(env, e->Iex.Unop.arg);
+         addInstr(env, PPC32Instr_Unary32(Pun_NOT,r_dst,r_src));
+         return r_dst;
       }
       case Iop_64HIto32: {
          HReg rHi, rLo;
@@ -1184,11 +1187,11 @@ static HReg iselIntExpr_R_wrk ( ISelEnv* env, IRExpr* e )
                            unop(Iop_64to32,
                                 binop(Iop_MullS32, bind(0), bind(1))));
             if (matchIRExpr(&mi,p_MullS32_then_64to32,e)) {
-               HReg dst      = newVRegI(env);
-               HReg src1     = iselIntExpr_R( env, mi.bindee[0] );
-               PPC32RI* src2 = mk_FitRI16_S(env, iselIntExpr_RI( env, mi.bindee[1] ));
-               addInstr(env, PPC32Instr_MulL(True, 0, dst, src1, src2));
-               return dst;
+               HReg r_dst         = newVRegI(env);
+               HReg r_srcL      = iselIntExpr_R( env, mi.bindee[0] );
+               PPC32RI* ri_srcR = mk_FitRI16_S(env, iselIntExpr_RI( env, mi.bindee[1] ));
+               addInstr(env, PPC32Instr_MulL(True, 0, r_dst, r_srcL, ri_srcR));
+               return r_dst;
             }
          }
 
@@ -1199,11 +1202,11 @@ static HReg iselIntExpr_R_wrk ( ISelEnv* env, IRExpr* e )
                            unop(Iop_64to32,
                                 binop(Iop_MullU32, bind(0), bind(1))));
             if (matchIRExpr(&mi,p_MullU32_then_64to32,e)) {
-               HReg dst      = newVRegI(env);
-               HReg src1     = iselIntExpr_R( env, mi.bindee[0] );
-               PPC32RI* src2 = mk_FitRI16_S(env, iselIntExpr_RI( env, mi.bindee[1] ));
-               addInstr(env, PPC32Instr_MulL(False, 0, dst, src1, src2));
-               return dst;
+               HReg r_dst         = newVRegI(env);
+               HReg r_srcL      = iselIntExpr_R( env, mi.bindee[0] );
+               PPC32RI* ri_srcR = mk_FitRI16_S(env, iselIntExpr_RI( env, mi.bindee[1] ));
+               addInstr(env, PPC32Instr_MulL(False, 0, r_dst, r_srcL, ri_srcR));
+               return r_dst;
             }
          }
 
@@ -1217,29 +1220,29 @@ static HReg iselIntExpr_R_wrk ( ISelEnv* env, IRExpr* e )
       }
       case Iop_16HIto8:
       case Iop_32HIto16: {
-         HReg dst   = newVRegI(env);
-         HReg src   = iselIntExpr_R(env, e->Iex.Unop.arg);
+         HReg r_dst = newVRegI(env);
+         HReg r_src = iselIntExpr_R(env, e->Iex.Unop.arg);
          UInt shift = e->Iex.Unop.op == Iop_16HIto8 ? 8 : 16;
-         addInstr(env, mk_sh32(env, Psh_SHR, dst, src, PPC32RI_Imm(shift)));
-         return dst;
+         addInstr(env, mk_sh32(env, Psh_SHR, r_dst, r_src, PPC32RI_Imm(shift)));
+         return r_dst;
       }
       case Iop_1Uto32:
       case Iop_1Uto8: {
-         HReg dst           = newVRegI(env);
+         HReg r_dst           = newVRegI(env);
          PPC32CondCode cond = iselCondCode(env, e->Iex.Unop.arg);
-         addInstr(env, PPC32Instr_Set32(cond,dst));
-         return dst;
+         addInstr(env, PPC32Instr_Set32(cond,r_dst));
+         return r_dst;
       }
       case Iop_1Sto8:
       case Iop_1Sto16:
       case Iop_1Sto32: {
          /* could do better than this, but for now ... */
-         HReg dst           = newVRegI(env);
+         HReg r_dst           = newVRegI(env);
          PPC32CondCode cond = iselCondCode(env, e->Iex.Unop.arg);
-         addInstr(env, PPC32Instr_Set32(cond,dst));
-         addInstr(env, mk_sh32(env, Psh_SHL, dst, dst, PPC32RI_Imm(31)));
-         addInstr(env, mk_sh32(env, Psh_SAR, dst, dst, PPC32RI_Imm(31)));
-         return dst;
+         addInstr(env, PPC32Instr_Set32(cond,r_dst));
+         addInstr(env, mk_sh32(env, Psh_SHL, r_dst, r_dst, PPC32RI_Imm(31)));
+         addInstr(env, mk_sh32(env, Psh_SAR, r_dst, r_dst, PPC32RI_Imm(31)));
+         return r_dst;
       }
 
 //..          case Iop_Ctz32: {
@@ -1251,10 +1254,10 @@ static HReg iselIntExpr_R_wrk ( ISelEnv* env, IRExpr* e )
 //..          }
       case Iop_Clz32: {
          /* Count leading zeroes. */
-         HReg dst = newVRegI(env);
-         HReg src = iselIntExpr_R(env, e->Iex.Unop.arg);
-         addInstr(env, PPC32Instr_Unary32(Pun_CLZ,dst,src));
-         return dst;
+         HReg r_dst = newVRegI(env);
+         HReg r_src = iselIntExpr_R(env, e->Iex.Unop.arg);
+         addInstr(env, PPC32Instr_Unary32(Pun_CLZ,r_dst,r_src));
+         return r_dst;
       }
 
 //..          case Iop_128to32: {
@@ -1283,10 +1286,10 @@ static HReg iselIntExpr_R_wrk ( ISelEnv* env, IRExpr* e )
    /* --------- GET --------- */
    case Iex_Get: {
       if (ty == Ity_I8 || ty == Ity_I16 || ty == Ity_I32) {
-         HReg dst = newVRegI(env);
+         HReg r_dst = newVRegI(env);
          PPC32AMode* am_src = PPC32AMode_IR(e->Iex.Get.offset, GuestStatePtr );
-         addInstr(env, PPC32Instr_Load( sizeofIRType(ty), False, dst, am_src ));
-         return dst;
+         addInstr(env, PPC32Instr_Load( sizeofIRType(ty), False, r_dst, am_src ));
+         return r_dst;
       }
       break;
    }
@@ -1306,7 +1309,7 @@ static HReg iselIntExpr_R_wrk ( ISelEnv* env, IRExpr* e )
 
    /* --------- CCALL --------- */
    case Iex_CCall: {
-      HReg    dst = newVRegI(env);
+      HReg    r_dst = newVRegI(env);
       vassert(ty == Ity_I32);
 
       /* be very restrictive for now.  Only 32/64-bit ints allowed
@@ -1318,16 +1321,16 @@ static HReg iselIntExpr_R_wrk ( ISelEnv* env, IRExpr* e )
       doHelperCall( env, False, NULL, e->Iex.CCall.cee, e->Iex.CCall.args );
 
       /* GPR3 now holds the destination address from Pin_Goto */
-      addInstr(env, mk_iMOVds_RR(dst, hregPPC32_GPR3()));
-      return dst;
+      addInstr(env, mk_iMOVds_RR(r_dst, hregPPC32_GPR3()));
+      return r_dst;
    }
       
    /* --------- LITERAL --------- */
    /* 32/16/8-bit literals */
    case Iex_Const: {
-      HReg dst    = newVRegI(env);
-      addInstr(env, mk_iMOVds_RRI(env, dst, iselIntExpr_RI ( env, e )));
-      return dst;
+      HReg r_dst    = newVRegI(env);
+      addInstr(env, mk_iMOVds_RRI(env, r_dst, iselIntExpr_RI ( env, e )));
+      return r_dst;
    }
 
    /* --------- MULTIPLEX --------- */
@@ -1338,14 +1341,14 @@ static HReg iselIntExpr_R_wrk ( ISelEnv* env, IRExpr* e )
          HReg r_cond;
          HReg rX     = iselIntExpr_R(env, e->Iex.Mux0X.exprX);
          PPC32RI* r0 = iselIntExpr_RI(env, e->Iex.Mux0X.expr0);
-         HReg dst    = newVRegI(env);
+         HReg r_dst    = newVRegI(env);
 
-         addInstr(env, mk_iMOVds_RR(dst,rX));
+         addInstr(env, mk_iMOVds_RR(r_dst,rX));
          r_cond = iselIntExpr_R(env, e->Iex.Mux0X.cond);
          addInstr(env, PPC32Instr_Cmp32(Pcmp_U, 7, r_cond, PPC32RI_Imm(0)));
          cc = mk_PPCCondCode( Pct_TRUE, Pcf_EQ );
-         addInstr(env, PPC32Instr_CMov32(cc,dst,r0));
-         return dst;
+         addInstr(env, PPC32Instr_CMov32(cc,r_dst,r0));
+         return r_dst;
       }
       break;
    }
@@ -1536,8 +1539,8 @@ static PPC32CondCode iselCondCode_wrk ( ISelEnv* env, IRExpr* e )
    /* pattern: 32to1(expr32) */
    DEFINE_PATTERN(p_32to1, unop(Iop_32to1,bind(0)));
    if (matchIRExpr(&mi,p_32to1,e)) {
-      HReg dst = iselIntExpr_R(env, mi.bindee[0]);
-      addInstr(env, PPC32Instr_Cmp32(Pcmp_U, 7, dst, PPC32RI_Imm(1)));
+      HReg r_dst = iselIntExpr_R(env, mi.bindee[0]);
+      addInstr(env, PPC32Instr_Cmp32(Pcmp_U, 7, r_dst, PPC32RI_Imm(1)));
       return mk_PPCCondCode( Pct_TRUE, Pcf_EQ );
    }
 
@@ -1669,9 +1672,9 @@ static PPC32CondCode iselCondCode_wrk ( ISelEnv* env, IRExpr* e )
 
    /* var */
    if (e->tag == Iex_Tmp) {
-      HReg src        = lookupIRTemp(env, e->Iex.Tmp.tmp);
+      HReg r_src      = lookupIRTemp(env, e->Iex.Tmp.tmp);
       HReg src_masked = newVRegI(env);
-      addInstr(env, PPC32Instr_Alu32(Palu_AND, src_masked, src, PPC32RI_Imm(1)));
+      addInstr(env, PPC32Instr_Alu32(Palu_AND, src_masked, r_src, PPC32RI_Imm(1)));
       addInstr(env, PPC32Instr_Cmp32(Pcmp_U, 7, src_masked, PPC32RI_Imm(1)));
       return mk_PPCCondCode( Pct_TRUE, Pcf_EQ );
    }
@@ -1803,16 +1806,16 @@ static void iselInt64Expr_wrk ( HReg* rHi, HReg* rLo, ISelEnv* env, IRExpr* e )
          /* get one operand into %r3, and the other into a R/I.
             Need to make an educated guess about which is better in
             which. */
-         HReg     tLo   = newVRegI(env);
-         HReg     tHi   = newVRegI(env);
-         Bool     syned = e->Iex.Binop.op == Iop_MullS32;
-         HReg     src1  = iselIntExpr_R(env, e->Iex.Binop.arg1);
+         HReg     tLo     = newVRegI(env);
+         HReg     tHi     = newVRegI(env);
+         Bool     syned   = e->Iex.Binop.op == Iop_MullS32;
+         HReg     r_srcL  = iselIntExpr_R(env, e->Iex.Binop.arg1);
 
 // CAB: could do better than this...
-         PPC32RI* src2 = PPC32RI_Reg(iselIntExpr_R(env, e->Iex.Binop.arg2));
+         PPC32RI* ri_srcR = PPC32RI_Reg(iselIntExpr_R(env, e->Iex.Binop.arg2));
 
-         addInstr(env, PPC32Instr_MulL(syned, 0, tLo, src1, src2));
-         addInstr(env, PPC32Instr_MulL(syned, 1, tHi, src1, src2));
+         addInstr(env, PPC32Instr_MulL(syned, 0, tLo, r_srcL, ri_srcR));
+         addInstr(env, PPC32Instr_MulL(syned, 1, tHi, r_srcL, ri_srcR));
          *rHi = tHi;
          *rLo = tLo;
          return;
@@ -3195,8 +3198,8 @@ static void iselStmt ( ISelEnv* env, IRStmt* stmt )
       vassert(tya == Ity_I32);
       am_dst = iselIntExpr_AMode(env, stmt->Ist.STle.addr);
       if (tyd == Ity_I8 || tyd == Ity_I16 || tyd == Ity_I32) {
-         HReg src = iselIntExpr_R(env, stmt->Ist.STle.data);
-         addInstr(env, PPC32Instr_Store(sizeofIRType(tyd),am_dst,src));
+         HReg r_src = iselIntExpr_R(env, stmt->Ist.STle.data);
+         addInstr(env, PPC32Instr_Store(sizeofIRType(tyd),am_dst,r_src));
          return;
       }
 //..       if (tyd == Ity_F64) {
@@ -3231,9 +3234,9 @@ static void iselStmt ( ISelEnv* env, IRStmt* stmt )
    case Ist_Put: {
       IRType ty = typeOfIRExpr(env->type_env, stmt->Ist.Put.data);
       if (ty == Ity_I8 || ty == Ity_I16 || ty == Ity_I32) {
-         HReg src = iselIntExpr_R(env, stmt->Ist.Put.data);
+         HReg r_src = iselIntExpr_R(env, stmt->Ist.Put.data);
          PPC32AMode* am_dst = PPC32AMode_IR(stmt->Ist.Put.offset, GuestStatePtr);
-         addInstr(env, PPC32Instr_Store( sizeofIRType(ty), am_dst, src ));
+         addInstr(env, PPC32Instr_Store( sizeofIRType(ty), am_dst, r_src ));
          return;
       }
 //..       if (ty == Ity_I64) {
@@ -3296,9 +3299,9 @@ static void iselStmt ( ISelEnv* env, IRStmt* stmt )
       IRTemp tmp = stmt->Ist.Tmp.tmp;
       IRType ty = typeOfIRTemp(env->type_env, tmp);
       if (ty == Ity_I32 || ty == Ity_I16 || ty == Ity_I8) {
-         HReg dst = lookupIRTemp(env, tmp);
-         HReg src = iselIntExpr_R(env, stmt->Ist.Tmp.data);
-         addInstr(env, mk_iMOVds_RR( dst, src ));
+         HReg r_dst = lookupIRTemp(env, tmp);
+         HReg r_src = iselIntExpr_R(env, stmt->Ist.Tmp.data);
+         addInstr(env, mk_iMOVds_RR( r_dst, r_src ));
          return;
       }
 //..       if (ty == Ity_I64) {
@@ -3311,8 +3314,8 @@ static void iselStmt ( ISelEnv* env, IRStmt* stmt )
 //..       }
       if (ty == Ity_I1) {
          PPC32CondCode cond = iselCondCode(env, stmt->Ist.Tmp.data);
-         HReg dst = lookupIRTemp(env, tmp);
-         addInstr(env, PPC32Instr_Set32(cond, dst));
+         HReg r_dst = lookupIRTemp(env, tmp);
+         addInstr(env, PPC32Instr_Set32(cond, r_dst));
          return;
       }
 //..       if (ty == Ity_F64) {
@@ -3381,14 +3384,14 @@ static void iselStmt ( ISelEnv* env, IRStmt* stmt )
 
    /* --------- EXIT --------- */
    case Ist_Exit: {
-      PPC32RI*      dst;
+      PPC32RI*      ri_dst;
       PPC32CondCode cc;
       if (stmt->Ist.Exit.dst->tag != Ico_U32)
          vpanic("isel_ppc32: Ist_Exit: dst is not a 32-bit value");
-      dst = iselIntExpr_RI(env, IRExpr_Const(stmt->Ist.Exit.dst));
-      cc  = iselCondCode(env,stmt->Ist.Exit.guard);
+      ri_dst = iselIntExpr_RI(env, IRExpr_Const(stmt->Ist.Exit.dst));
+      cc     = iselCondCode(env,stmt->Ist.Exit.guard);
       addInstr(env, PPC32Instr_RdWrLR(True, env->savedLR));
-      addInstr(env, PPC32Instr_Goto(stmt->Ist.Exit.jk, cc, dst));
+      addInstr(env, PPC32Instr_Goto(stmt->Ist.Exit.jk, cc, ri_dst));
       return;
    }