Enable GRF spill/fill space compression by default for 3D.

kychendev · igcbot · commit 1657e913ac3e · 2024-02-26T19:12:18.000+01:00
Remove the spill size heuristic for 3D workloads so spill space
compression is always enabled by default.
diff --git a/visa/GraphColor.cpp b/visa/GraphColor.cpp
@@ -10707,8 +10707,7 @@ GlobalRA::insertSpillCode(bool enableSpillSpaceCompression,
   if (getIterNo() == 0 && enableSpillSpaceCompression &&
       kernel.getInt32KernelAttr(Attributes::ATTR_Target) == VISA_3D &&
       !hasStackCall) {
-    enableSpillSpaceCompression = spillSpaceCompression(
-        computeSpillSize(coloring.getSpilledLiveRanges()), globalScratchOffset);
+    enableSpillSpaceCompression = true;
   }
 
   startTimer(TimerID::SPILL);

Original file line number	Diff line number	Diff line change
`@@ -10707,8 +10707,7 @@ GlobalRA::insertSpillCode(bool enableSpillSpaceCompression,`
`10707`	`10707`	`if (getIterNo() == 0 && enableSpillSpaceCompression &&`
`10708`	`10708`	`kernel.getInt32KernelAttr(Attributes::ATTR_Target) == VISA_3D &&`
`10709`	`10709`	`!hasStackCall) {`
`10710`		`- enableSpillSpaceCompression = spillSpaceCompression(`
`10711`		`- computeSpillSize(coloring.getSpilledLiveRanges()), globalScratchOffset);`
	`10710`	`+ enableSpillSpaceCompression = true;`
`10712`	`10711`	`}`
`10713`	`10712`
`10714`	`10713`	`startTimer(TimerID::SPILL);`