- Track register pressure more accurately when there are

partially-written local variables.

- Fixes a crash in liveness

Change-Id: I7d6f14d9c7e00c9faf347b7f9707129c192c74a6

Author: sys-d3djenkins

visa/LocalScheduler/G4_Sched.cpp

@@ -303,16 +303,11 @@ struct RegisterPressure
         }
     }
 
-    RegisterPressure(const RegisterPressure& other) = delete;
-    RegisterPressure& operator=(RegisterPressure& other) = delete;
-
     void init()
     {
         p2a = new PointsToAnalysis(kernel.Declares, kernel.fg.getNumBB());
         p2a->doPointsToAnalysis(kernel.fg);
         gra = new GlobalRA(kernel, kernel.fg.builder->phyregpool, *p2a);
-        // To properly track liveness for partially-written local variables.
-        gra->markGraphBlockLocalVars(/*doLocalRA*/false);
         liveness = new LivenessAnalysis(*gra, G4_GRF | G4_ADDRESS | G4_INPUT | G4_FLAG);
         liveness->computeLiveness(true);
         rpe = new RPE(*gra, liveness);
@@ -387,6 +382,11 @@ struct RegisterPressure
         }
         std::cerr << "\n\n";
     }
+    private:
+        // Private copy ctor, assignment operator to prevent shallow copy that
+        // could lead to double free.
+        RegisterPressure(const RegisterPressure& other);
+        RegisterPressure& operator=(RegisterPressure& other);
 };
 
 struct SchedConfig

visa/RPE.cpp

@@ -42,12 +42,9 @@ namespace vISA
     void RPE::run()
     {
         startTimer(TIMER_RPE);
-        if (liveAnalysis->getNumSelectedVar() > 0)
+        for (auto& bb : gra.kernel.fg.BBs)
         {
-            for (auto& bb : gra.kernel.fg.BBs)
-            {
-                runBB(bb);
-            }
+            runBB(bb);
         }
         stopTimer(TIMER_RPE);
     }