Changeset 4020 in CLRX


Ignore:
Timestamp:
Apr 18, 2018, 2:54:59 PM (6 months ago)
Author:
matszpk
Message:

CLRadeonExtender: AsmRegAlloc?: Do nothing if no SSA replaces (applySSAReplaces). Add testcase with empty SSA replaces.

Location:
CLRadeonExtender/trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • CLRadeonExtender/trunk/amdasm/AsmRegAlloc.cpp

    r4016 r4020  
    457457void AsmRegAllocator::applySSAReplaces()
    458458{
     459    if (ssaReplacesMap.empty())
     460        return; // do nothing
     461   
    459462    /* prepare SSA id replaces */
    460463    struct MinSSAGraphNode
     
    675678            }
    676679        }
     680   
     681    // clear ssa replaces
     682    ssaReplacesMap.clear();
    677683}
    678684
  • CLRadeonExtender/trunk/tests/amdasm/AsmRegAlloc2.cpp

    r4004 r4020  
    822822                }, false, false, false }
    823823        }
    824     }
     824    },
     825    {   // 5 - empty
     826        {
     827            { 0, 4,
     828                { },
     829                {
     830                    { { "sa", 2 }, SSAInfo(0, 1, 1, 1, 1, true) }
     831                }, false, false, false }
     832        },
     833        { }, // empty SSA replaces
     834        {
     835            { 0, 4,
     836                { },
     837                {
     838                    { { "sa", 2 }, SSAInfo(0, 1, 1, 1, 1, true) }
     839                }, false, false, false }
     840        }
     841    }   
    825842};
    826843
Note: See TracChangeset for help on using the changeset viewer.