[Autobackout][FuncReg]Revert of change: 7613e49

 Revert: Enable pre RA scheduling for ACC register pressure

rever Enable pre RA scheduling for ACC register pressure

Author: bcheng0127

visa/HWCaps.inc

@@ -762,6 +762,11 @@ SPDX-License-Identifier: MIT
         return false;
     }
 
+    bool enablePreSchedACC() const
+    {
+        return false;
+    }
+
     bool hasDoubleAcc() const
     {
         return getOption(vISA_hasDoubleAcc);

visa/LocalScheduler/G4_Sched.cpp

@@ -2623,6 +2623,11 @@ bool preRA_ACC_Scheduler::run()
     AccSubPass accSub(*kernel.fg.builder, kernel);
 
     for (auto bb : kernel.fg) {
+        if (bb->size() < SMALL_BLOCK_SIZE || bb->size() > LARGE_BLOCK_SIZE) {
+            //Skip small and large blocks.
+            continue;
+        }
+
         preDDD ddd(mem, kernel, bb);
         SchedConfig config(0);
         BB_ACC_Scheduler S(kernel, ddd);

visa/Optimizer.cpp

@@ -2089,12 +2089,12 @@ int Optimizer::optimization()
     // HW workaround before RA
     runPass(PI_preRA_HWWorkaround);
 
-    if (builder.enableACCBeforRA() && kernel.getOption(vISA_PreSchedForAcc))
+    if (builder.enableACCBeforRA() && builder.enablePreSchedACC())
     {
         runPass(PI_ACCSchedule);
     }
 
-    if (builder.enableACCBeforRA() && !kernel.getOption(vISA_PreSchedForAcc))
+    if (builder.enableACCBeforRA() && !builder.enablePreSchedACC())
     {
         runPass(PI_accSubBeforeRA);
     }
@@ -2137,7 +2137,7 @@ int Optimizer::optimization()
         runPass(PI_localSchedule);
     }
 
-    if (!builder.enableACCBeforRA() && !kernel.getOption(vISA_PreSchedForAcc))
+    if (!builder.enableACCBeforRA() && !builder.enablePreSchedACC())
     {
         runPass(PI_expandMulPostSchedule);
 

visa/include/VISAOptionsDefs.h

@@ -82,8 +82,7 @@ DEF_VISA_OPTION(vISA_EnableSplitVariables,  ET_BOOL, "-noSplitVariables", UNUSED
 DEF_VISA_OPTION(vISA_ChangeMoveType,        ET_BOOL, "-ALTMode",       UNUSED, true)
 DEF_VISA_OPTION(vISA_accSubstitution,       ET_BOOL, "-noAccSub",          UNUSED, true)
 DEF_VISA_OPTION(vISA_accSubBeforeRA,        ET_BOOL, "-noAccSubBRA",          UNUSED, true)
-DEF_VISA_OPTION(vISA_PreSchedForAcc,   ET_BOOL,  "-preSchedForAcc", UNUSED, false)
-DEF_VISA_OPTION(vISA_DepthFirstPreSchedForAcc,   ET_BOOL,  "-depthFirstPreSchedForAcc", UNUSED, false)
+DEF_VISA_OPTION(vISA_PreSchedForAcc,   ET_BOOL,  "-preSchedForAcc", UNUSED, true)
 DEF_VISA_OPTION(vISA_EnableGatherWithImm,      ET_BOOL, "-gatherWithImm",        UNUSED, 0)
 DEF_VISA_OPTION(vISA_doAccSubAfterSchedule, ET_BOOL, "-accSubPostSchedule",    UNUSED, true)
 DEF_VISA_OPTION(vISA_localizationForAccSub, ET_BOOL, "-localizeForACC",    UNUSED, false)