mirrors/Ryujinx
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix LiveInterval.Split (#1660)

Browse source 
Before when splitting intervals, the end of the range would be included
in the split check, this can produce empty ranges in the child split.

This in turn can affect spilling decisions since the child split will
have a different start position and this empty range will get a register
and move to the active set for a brief moment.

For example:

  A = [153, 172[; [1899, 1916[; [1991, 2010[; [2397, 2414[; ...

  Split(A, 1916)

  A0 = [153, 172[; [1899, 1916[
  A1 = [1916, 1916[; [1991, 2010[; [2397, 2414[; ...
This commit is contained in:
FICTURE7 2020-11-05 06:09:45 +04:00 • committed by Matt Heins
commit 91f543a8f9
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
ARMeilleure/CodeGen/RegisterAllocators/LiveInterval.cs
View file

@ -287,7 +287,7 @@ namespace ARMeilleure.CodeGen.RegisterAllocators
{
LiveRange range = _ranges[splitIndex];
if (position > range.Start && position <= range.End)
if (position > range.Start && position < range.End)
{
right._ranges.Add(new LiveRange(position, range.End));