diff options
| author | Evandro Menezes <e.menezes@samsung.com> | 2019-10-02 19:44:53 +0000 |
|---|---|---|
| committer | Evandro Menezes <e.menezes@samsung.com> | 2019-10-02 19:44:53 +0000 |
| commit | e139a73c5fc1fc7139b08c7ff3adf3a29e1f7dc8 (patch) | |
| tree | 1c955a97bd19bec5f1199e94a93b52fb78184263 | |
| parent | a80b6c15425f82521c624ff24c5c0a34cd534d54 (diff) | |
| download | bcm5719-llvm-e139a73c5fc1fc7139b08c7ff3adf3a29e1f7dc8.tar.gz bcm5719-llvm-e139a73c5fc1fc7139b08c7ff3adf3a29e1f7dc8.zip | |
[TableGen] Improve error reporting of overlapping definitions (NFC)
llvm-svn: 373514
| -rw-r--r-- | llvm/utils/TableGen/CodeGenSchedule.cpp | 20 |
1 files changed, 14 insertions, 6 deletions
diff --git a/llvm/utils/TableGen/CodeGenSchedule.cpp b/llvm/utils/TableGen/CodeGenSchedule.cpp index cb05f78fba4..f12d7d484a8 100644 --- a/llvm/utils/TableGen/CodeGenSchedule.cpp +++ b/llvm/utils/TableGen/CodeGenSchedule.cpp @@ -1083,9 +1083,13 @@ void CodeGenSchedModels::createInstRWClass(Record *InstRWDef) { if (RWD->getValueAsDef("SchedModel") == RWModelDef && RWModelDef->getValueAsBit("FullInstRWOverlapCheck")) { for (Record *Inst : InstDefs) { - PrintFatalError(InstRWDef->getLoc(), "Overlapping InstRW def " + - Inst->getName() + " also matches " + - RWD->getValue("Instrs")->getValue()->getAsString()); + PrintFatalError + (InstRWDef->getLoc(), + "Overlapping InstRW definition for \"" + + Inst->getName() + + "\" also matches previous \"" + + RWD->getValue("Instrs")->getValue()->getAsString() + + "\"."); } } } @@ -1115,9 +1119,13 @@ void CodeGenSchedModels::createInstRWClass(Record *InstRWDef) { for (Record *OldRWDef : SchedClasses[OldSCIdx].InstRWs) { if (OldRWDef->getValueAsDef("SchedModel") == RWModelDef) { for (Record *InstDef : InstDefs) { - PrintFatalError(OldRWDef->getLoc(), "Overlapping InstRW def " + - InstDef->getName() + " also matches " + - OldRWDef->getValue("Instrs")->getValue()->getAsString()); + PrintFatalError + (InstRWDef->getLoc(), + "Overlapping InstRW definition for \"" + + InstDef->getName() + + "\" also matches previous \"" + + OldRWDef->getValue("Instrs")->getValue()->getAsString() + + "\"."); } } assert(OldRWDef != InstRWDef && |
