Changeset 3972 in CLRX


Ignore:
Timestamp:
Apr 8, 2018, 12:21:09 PM (3 months ago)
Author:
matszpk
Message:

CLRadeonExtender: AsmRegAlloc?: Fixed skipping third recursion pass. Fixed last testcase.

Location:
CLRadeonExtender/trunk
Files:
2 edited

Legend:

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

    r3971 r3972  
    15601560            // if already added
    15611561            beforeFirstAccess = false;
    1562            
     1562       
    15631563        rdata.origRbwSSAIdMap.insert({ ssaEntry.first,
    15641564                        ssaEntry.second.ssaIdBefore }).second;
     
    18601860           
    18611861            // process current block
    1862             //if (/*cachedRdata != nullptr &&*/
    1863                 //visited[entry.blockIndex] && flowStack.size() > 1)
    18641862            const RoutineData* cachedRdata = nullptr;
    18651863           
     
    22862284            else
    22872285            {
     2286                // TODO: subroutToCache do not cache calls
    22882287                // handle caching for res second point
    22892288                cblocksToCache.increase(entry.blockIndex);
     
    23572356                        curSSAIdMapStateMap.insert({ nextBlock.index,  curSSAIdMap });
    23582357                    }
    2359                     else
     2358                    else if (entry.blockIndex.pass==1)
    23602359                    {
    23612360                        entry.nextIndex++;
  • CLRadeonExtender/trunk/tests/amdasm/AsmRegAllocCase3.cpp

    r3967 r3972  
    15381538                {
    15391539                    { { "sa", 0 }, SSAInfo(0, SIZE_MAX, 1, SIZE_MAX, 0, true) },
    1540                     { { "sa", 2 }, SSAInfo(3, 5, 5, 5, 1, true) },
     1540                    { { "sa", 2 }, SSAInfo(1, 5, 5, 5, 1, true) },
    15411541                    { { "sa", 3 }, SSAInfo(2, 5, 5, 5, 1, true) },
    15421542                    { { "sa", 6 }, SSAInfo(1, 4, 4, 4, 1, true) }
     
    15941594        },
    15951595        {   // SSA replaces (good ???)
    1596             { { "sa", 2 }, { { 3, 2 }, { 4, 2 }, { 4, 3 }, { 2, 1 }, { 3, 1 } } },
     1596            { { "sa", 2 }, { { 3, 2 }, { 4, 2 }, { 3, 1 }, { 4, 1 }, { 2, 1 } } },
    15971597            { { "sa", 3 }, { { 3, 2 }, { 4, 2 }, { 2, 1 }, { 3, 1 } } },
    15981598            { { "sa", 6 }, { { 2, 1 }, { 3, 1 } } }
Note: See TracChangeset for help on using the changeset viewer.