[Autobackout][FuncReg]Revert of change: 59b5f50

bcheng0127 · igcbot · commit cb1446b1e77c · 2023-12-16T15:09:01.000+01:00
Enable GRF spill/fill space compression by default for 3D

Remove the heuristic for the 3D
diff --git a/visa/GraphColor.cpp b/visa/GraphColor.cpp
@@ -9824,7 +9824,8 @@ GlobalRA::insertSpillCode(bool enableSpillSpaceCompression,
   if (getIterNo() == 0 && enableSpillSpaceCompression &&
       kernel.getInt32KernelAttr(Attributes::ATTR_Target) == VISA_3D &&
       !hasStackCall) {
-    enableSpillSpaceCompression = true;
+    enableSpillSpaceCompression = spillSpaceCompression(
+        computeSpillSize(coloring.getSpilledLiveRanges()), globalScratchOffset);
   }
 
   startTimer(TimerID::SPILL);

Original file line number	Diff line number	Diff line change
`@@ -9824,7 +9824,8 @@ GlobalRA::insertSpillCode(bool enableSpillSpaceCompression,`
`9824`	`9824`	`if (getIterNo() == 0 && enableSpillSpaceCompression &&`
`9825`	`9825`	`kernel.getInt32KernelAttr(Attributes::ATTR_Target) == VISA_3D &&`
`9826`	`9826`	`!hasStackCall) {`
`9827`		`- enableSpillSpaceCompression = true;`
	`9827`	`+ enableSpillSpaceCompression = spillSpaceCompression(`
	`9828`	`+ computeSpillSize(coloring.getSpilledLiveRanges()), globalScratchOffset);`
`9828`	`9829`	`}`
`9829`	`9830`
`9830`	`9831`	`startTimer(TimerID::SPILL);`