re-eanble read-modify-write for spills not inside divergent control flow.

weiyu-chen · sys_zuul · commit a69efe8cc0e9 · 2020-06-04T18:42:33.000-07:00
Change-Id: Ifc7b20525b7bfb5035779c5095b1d58a55209a77
diff --git a/visa/SpillManagerGMRF.cpp b/visa/SpillManagerGMRF.cpp
@@ -2535,7 +2535,7 @@ SpillManagerGRF::shouldPreloadSpillRange(
 
     if (isPartialRegion(spilledRangeRegion, execSize) ||
         isUnalignedRegion(spilledRangeRegion, execSize) ||
-        instContext->isPartialWriteForSpill(parentBB->isDivergent()))
+        instContext->isPartialWriteForSpill(!parentBB->isAllLaneActive()))
     {
 #if 0
         // special check for scalar variables: no need for pre-fill if instruction is not predicated

Original file line number	Diff line number	Diff line change
`@@ -2535,7 +2535,7 @@ SpillManagerGRF::shouldPreloadSpillRange(`
`2535`	`2535`
`2536`	`2536`	`if (isPartialRegion(spilledRangeRegion, execSize) \|\|`
`2537`	`2537`	`isUnalignedRegion(spilledRangeRegion, execSize) \|\|`
`2538`		`- instContext->isPartialWriteForSpill(parentBB->isDivergent()))`
	`2538`	`+ instContext->isPartialWriteForSpill(!parentBB->isAllLaneActive()))`
`2539`	`2539`	`{`
`2540`	`2540`	`#if 0`
`2541`	`2541`	`// special check for scalar variables: no need for pre-fill if instruction is not predicated`