Changes in code.

Change-Id: I1a9a4f7695ace86c7cd158f2b296988077bb991d

Author: mkarigan

visa/G4Verifier.cpp

@@ -796,18 +796,6 @@ void G4Verifier::verifyOpnd(G4_Operand* opnd, G4_INST* inst)
                     }
                 }
             }
-
-            // if src0 is V/UV/VF imm, dst must be 16 byte aligned.
-            if (inst->opcode() == G4_mov && IS_VTYPE(inst->getSrc(0)->getType()))
-            {
-                auto dst = inst->getDst();
-                // should we assert if dst is not phyReg assigned?
-                bool dstIsAssigned = dst->getBase()->isRegVar() && dst->getBase()->asRegVar()->isPhyRegAssigned();
-                if (dstIsAssigned && dst->getLinearizedStart() % 16 != 0)
-                {
-                    assert(false && "destination of move instruction with V/VF imm is not 16-byte aligned");
-                }
-            }
         }
     }
 }

visa/LocalRA.cpp

@@ -2311,29 +2311,43 @@ bool PhyRegsLocalRA::findFreeSingleReg(int regIdx, G4_SubReg_Align subalign, int
             found = true;
         }
     }
-    else
+    else if (subalign == Eight_Word || subalign == Four_Word)
     {
-        // ToDo: check if dynamic step size has compile time impact
-        int step = 1;
-        switch (subalign)
+        for (int j = 0; j < (NUM_WORDS_PER_GRF - size + 1) && found == false; j += 4)
         {
-            case Eight_Word: step = 8; break;
-            case Four_Word: step = 4; break;
-            case Even_Word: step = 2; break;
-            case Any: step = 1; break;
-            default:
-                assert("unexpected alignment");
+            if (isWordBusy(regIdx, j, size) == false)
+            {
+                subregnum = j;
+                found = true;
+            }
         }
-        for (int j = 0; j + size <= NUM_WORDS_PER_GRF; j += step)
+    }
+    else if (subalign == Even_Word)
+    {
+        for (int j = 0; j < (NUM_WORDS_PER_GRF - size + 1) && found == false; j += 2)
         {
-            if (!isWordBusy(regIdx, j, size))
+            if (isWordBusy(regIdx, j, size) == false)
             {
                 subregnum = j;
                 found = true;
-                break;
             }
         }
     }
+    else if (subalign == Any)
+    {
+        for (int j = 0; j < (NUM_WORDS_PER_GRF - size) && found == false; j++)
+        {
+            if (isWordBusy(regIdx, j, size) == false)
+            {
+                subregnum = j;
+                found = true;
+            }
+        }
+    }
+    else
+    {
+        ASSERT_USER(false, "Dont know how to allocate this sub-alignment");
+    }
 
     if (found)
     {