Changeset 3337 in CLRX


Ignore:
Timestamp:
Sep 8, 2017, 7:24:26 AM (10 months ago)
Author:
matszpk
Message:

CLRadeonExtender: AsmFormats?: Remove obsolete skipSpacesAtEnd. Add checking garbages at doKCodeEnd.

Location:
CLRadeonExtender/trunk/amdasm
Files:
4 edited

Legend:

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

    r3333 r3337  
    527527{
    528528    Assembler& asmr = handler.assembler;
    529     const char* end = asmr.line + asmr.lineSize;
    530     skipSpacesToEnd(linePtr, end);
     529    if (handler.getDriverVersion() < 191205)
     530    {
     531        asmr.printError(pseudoOpPlace, "Global Data allowed only for new binary format");
     532        return;
     533    }
    531534    if (!checkGarbagesAtEnd(asmr, linePtr))
    532535        return;
    533     if (handler.getDriverVersion() < 191205)
    534     {
    535         asmr.printError(pseudoOpPlace, "Global Data allowed only for new binary format");
    536         return;
    537     }
    538536   
    539537    if (handler.rodataSection==ASMSECT_NONE)
     
    551549{
    552550    Assembler& asmr = handler.assembler;
    553     const char* end = asmr.line + asmr.lineSize;
    554     skipSpacesToEnd(linePtr, end);
     551   
     552    if (handler.getDriverVersion() < 191205)
     553    {
     554        asmr.printError(pseudoOpPlace, "Global RWData allowed only for new binary format");
     555        return;
     556    }
    555557    if (!checkGarbagesAtEnd(asmr, linePtr))
    556558        return;
    557    
    558     if (handler.getDriverVersion() < 191205)
    559     {
    560         asmr.printError(pseudoOpPlace, "Global RWData allowed only for new binary format");
    561         return;
    562     }
    563559   
    564560    if (handler.dataSection==ASMSECT_NONE)
     
    577573    Assembler& asmr = handler.assembler;
    578574    const char* end = asmr.line + asmr.lineSize;
    579     skipSpacesToEnd(linePtr, end);
    580575   
    581576    if (handler.getDriverVersion() < 191205)
     
    633628{
    634629    Assembler& asmr = handler.assembler;
    635     const char* end = asmr.line + asmr.lineSize;
    636     skipSpacesToEnd(linePtr, end);
    637     if (!checkGarbagesAtEnd(asmr, linePtr))
    638         return;
    639    
    640630    if (handler.getDriverVersion() < 191205)
    641631    {
     
    649639        return;
    650640    }
     641    if (!checkGarbagesAtEnd(asmr, linePtr))
     642        return;
    651643   
    652644    if (handler.samplerInitSection==ASMSECT_NONE)
     
    13431335        return;
    13441336    }
     1337    if (!checkGarbagesAtEnd(asmr, linePtr))
     1338        return;
    13451339   
    13461340    AmdCL2KernelConfig& config = handler.output.kernels[asmr.currentKernel].config;
     
    13591353{
    13601354    Assembler& asmr = handler.assembler;
    1361     const char* end = asmr.line + asmr.lineSize;
    13621355   
    13631356    if (asmr.currentKernel==ASMKERN_GLOBAL || asmr.currentKernel==ASMKERN_INNER)
     
    13731366    }
    13741367   
    1375     skipSpacesToEnd(linePtr, end);
    13761368    if (!checkGarbagesAtEnd(asmr, linePtr))
    13771369        return;
     
    13921384{
    13931385    Assembler& asmr = handler.assembler;
    1394     const char* end = asmr.line + asmr.lineSize;
    13951386   
    13961387    if (asmr.currentKernel==ASMKERN_GLOBAL || asmr.currentKernel==ASMKERN_INNER)
     
    14111402    }
    14121403   
    1413     skipSpacesToEnd(linePtr, end);
    14141404    if (!checkGarbagesAtEnd(asmr, linePtr))
    14151405        return;
     
    14301420{
    14311421    Assembler& asmr = handler.assembler;
    1432     const char* end = asmr.line + asmr.lineSize;
    14331422   
    14341423    if (asmr.currentKernel==ASMKERN_GLOBAL || asmr.currentKernel==ASMKERN_INNER)
     
    14441433    }
    14451434   
    1446     skipSpacesToEnd(linePtr, end);
    14471435    if (!checkGarbagesAtEnd(asmr, linePtr))
    14481436        return;
     
    14631451{
    14641452    Assembler& asmr = handler.assembler;
    1465     const char* end = asmr.line + asmr.lineSize;
    14661453   
    14671454    if (asmr.currentKernel==ASMKERN_GLOBAL || asmr.currentKernel==ASMKERN_INNER)
     
    14821469    }
    14831470   
    1484     skipSpacesToEnd(linePtr, end);
    14851471    if (!checkGarbagesAtEnd(asmr, linePtr))
    14861472        return;
     
    15011487{
    15021488    Assembler& asmr = handler.assembler;
    1503     const char* end = asmr.line + asmr.lineSize;
    1504    
    15051489    if (asmr.currentKernel==ASMKERN_GLOBAL || asmr.currentKernel==ASMKERN_INNER)
    15061490    {
     
    15221506    }
    15231507   
    1524     skipSpacesToEnd(linePtr, end);
    15251508    if (!checkGarbagesAtEnd(asmr, linePtr))
    15261509        return;
  • CLRadeonExtender/trunk/amdasm/AsmAmdFormat.cpp

    r3333 r3337  
    364364{
    365365    Assembler& asmr = handler.assembler;
    366     const char* end = asmr.line + asmr.lineSize;
    367     skipSpacesToEnd(linePtr, end);
    368366    if (!checkGarbagesAtEnd(asmr, linePtr))
    369367        return;
     
    383381{
    384382    Assembler& asmr = handler.assembler;
    385     const char* end = asmr.line + asmr.lineSize;
    386    
    387383    if (asmr.currentKernel==ASMKERN_GLOBAL)
    388384    {
     
    397393    }
    398394   
    399     skipSpacesToEnd(linePtr, end);
    400395    if (!checkGarbagesAtEnd(asmr, linePtr))
    401396        return;
     
    416411{
    417412    Assembler& asmr = handler.assembler;
    418     const char* end = asmr.line + asmr.lineSize;
    419    
    420413    if (asmr.currentKernel==ASMKERN_GLOBAL)
    421414    {
     
    423416        return;
    424417    }
    425     skipSpacesToEnd(linePtr, end);
    426418    AsmAmdHandler::Kernel& kernel = *handler.kernelStates[asmr.currentKernel];
    427419    if (kernel.metadataSection!=ASMSECT_NONE || kernel.headerSection!=ASMSECT_NONE ||
     
    529521{
    530522    Assembler& asmr = handler.assembler;
    531     const char* end = asmr.line + asmr.lineSize;
    532    
    533523    if (asmr.currentKernel==ASMKERN_GLOBAL)
    534524    {
     
    543533    }
    544534   
    545     skipSpacesToEnd(linePtr, end);
    546535    if (!checkGarbagesAtEnd(asmr, linePtr))
    547536        return;
  • CLRadeonExtender/trunk/amdasm/AsmGalliumFormat.cpp

    r3333 r3337  
    474474{
    475475    Assembler& asmr = handler.assembler;
    476     const char* end = asmr.line + asmr.lineSize;
    477476    if (handler.sections[asmr.currentSection].type != AsmSectionType::CONFIG)
    478477    {
     
    486485        return;
    487486    }
    488     skipSpacesToEnd(linePtr, end);
    489487    if (!checkGarbagesAtEnd(asmr, linePtr))
    490488        return;
     
    531529{
    532530    Assembler& asmr = handler.assembler;
    533     const char* end = asmr.line + asmr.lineSize;
    534     skipSpacesToEnd(linePtr, end);
    535531    if (!checkGarbagesAtEnd(asmr, linePtr))
    536532        return;
     
    967963{
    968964    Assembler& asmr = handler.assembler;
    969     const char* end = asmr.line + asmr.lineSize;
    970965    if (handler.sections[asmr.currentSection].type != AsmSectionType::CONFIG)
    971966    {
     
    973968        return;
    974969    }
    975     skipSpacesToEnd(linePtr, end);
    976970    if (!checkGarbagesAtEnd(asmr, linePtr))
    977971        return;
     
    11611155{
    11621156    Assembler& asmr = handler.assembler;
    1163     const char* end = asmr.line + asmr.lineSize;
    11641157    if (handler.sections[asmr.currentSection].type != AsmSectionType::CONFIG)
    11651158    {
     
    11731166        return;
    11741167    }
    1175     skipSpacesToEnd(linePtr, end);
    11761168    if (!checkGarbagesAtEnd(asmr, linePtr))
    11771169        return;
     
    13561348        return;
    13571349    }
     1350    if (!checkGarbagesAtEnd(asmr, linePtr))
     1351        return;
     1352   
    13581353    updateKCodeSel(handler, handler.kcodeSelection);
    13591354    std::vector<cxuint> oldKCodeSel = handler.kcodeSelection;
  • CLRadeonExtender/trunk/amdasm/AsmROCmFormat.cpp

    r3333 r3337  
    347347{
    348348    Assembler& asmr = handler.assembler;
    349     const char* end = asmr.line + asmr.lineSize;
    350349    if (asmr.currentKernel==ASMKERN_GLOBAL)
    351350    {
     
    354353    }
    355354   
    356     skipSpacesToEnd(linePtr, end);
    357355    if (!checkGarbagesAtEnd(asmr, linePtr))
    358356        return;
     
    753751{
    754752    Assembler& asmr = handler.assembler;
    755     const char* end = asmr.line + asmr.lineSize;
    756753   
    757754    if (asmr.currentKernel==ASMKERN_GLOBAL ||
     
    762759    }
    763760   
    764     skipSpacesToEnd(linePtr, end);
    765761    if (!checkGarbagesAtEnd(asmr, linePtr))
    766762        return;
     
    11261122        return;
    11271123    }
     1124    if (!checkGarbagesAtEnd(asmr, linePtr))
     1125        return;
     1126   
    11281127    updateKCodeSel(handler, handler.kcodeSelection);
    11291128    std::vector<cxuint> oldKCodeSel = handler.kcodeSelection;
Note: See TracChangeset for help on using the changeset viewer.