Changeset 3602 in CLRX


Ignore:
Timestamp:
Jan 9, 2018, 9:41:15 PM (11 months ago)
Author:
matszpk
Message:

CLRadeonExtender: AsmRegAlloc?: Revert resolveSSAConflicts (add SSAReplaces only if exists previous SSA will be read, do not join two SSA ids in two various writes).

Location:
CLRadeonExtender/trunk
Files:
2 edited

Legend:

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

    r3601 r3602  
    543543                        { entry.blockIndex, false } });
    544544                   
    545                     if (res.second)
     545                    if (res.second && sinfo.readBeforeWrite)
    546546                    {
    547547                        // resolve conflict for this variable ssaId>
     
    551551                            // found, resolve by set ssaIdLast
    552552                            for (size_t ssaId: it->second)
    553                             {
    554                                 if (sinfo.readBeforeWrite && ssaId > sinfo.ssaIdBefore)
     553                                if (ssaId > sinfo.ssaIdBefore)
    555554                                    insertReplace(replacesMap, sentry.first, ssaId,
    556555                                                sinfo.ssaIdBefore);
    557                                 else if (ssaId > sinfo.ssaIdFirst)
    558                                     insertReplace(replacesMap, sentry.first, ssaId,
    559                                                 sinfo.ssaIdFirst);
    560                             }
    561556                        res.first->second.handled = true;
    562557                    }
  • CLRadeonExtender/trunk/tests/amdasm/AsmRegAlloc.cpp

    r3601 r3602  
    12921292        {   // SSA replaces
    12931293            { { "sa", 0 }, { { 2, 1 } } },
    1294             { { "va", 0 }, { { 4, 2 } } },
    12951294            { { "va", 1 }, { { 2, 1 } } }
    12961295        },
     
    14241423            { { "sa", 0 }, { { 6, 2 }, { 8, 2 } } },
    14251424            { { "sa", 1 }, { { 7, 1 }, { 8, 1 } } },
    1426             { { "sa", 2 }, { { 9, 2 }, { 6, 2 } } },
    1427             { { "sa", 3 }, { { 5, 2 } } },
    14281425            { { "sa", 4 }, { { 4, 1 } } }
    14291426        },
Note: See TracChangeset for help on using the changeset viewer.