[TableGen] CodeGenMapTable - Don't dereference a dyn_cast result. NFCI.
The static analyzer is warning about potential null dereferences of dyn_cast<> results - in these cases we can safely use cast<> directly as we know that these cases should all be the correct type, which is why its working atm and anyway cast<> will assert if they aren't. llvm-svn: 372146
This commit is contained in:
parent
a9a27d1ded
commit
c191c24314
|
@ -132,7 +132,7 @@ public:
|
||||||
MapRec->getName() + "' has empty " + "`ValueCols' field!");
|
MapRec->getName() + "' has empty " + "`ValueCols' field!");
|
||||||
|
|
||||||
for (Init *I : ColValList->getValues()) {
|
for (Init *I : ColValList->getValues()) {
|
||||||
ListInit *ColI = dyn_cast<ListInit>(I);
|
auto *ColI = cast<ListInit>(I);
|
||||||
|
|
||||||
// Make sure that all the sub-lists in 'ValueCols' have same number of
|
// Make sure that all the sub-lists in 'ValueCols' have same number of
|
||||||
// elements as the fields in 'ColFields'.
|
// elements as the fields in 'ColFields'.
|
||||||
|
@ -521,7 +521,7 @@ static void emitEnums(raw_ostream &OS, RecordKeeper &Records) {
|
||||||
unsigned ListSize = List->size();
|
unsigned ListSize = List->size();
|
||||||
|
|
||||||
for (unsigned j = 0; j < ListSize; j++) {
|
for (unsigned j = 0; j < ListSize; j++) {
|
||||||
ListInit *ListJ = dyn_cast<ListInit>(List->getElement(j));
|
auto *ListJ = cast<ListInit>(List->getElement(j));
|
||||||
|
|
||||||
if (ListJ->size() != ColFields->size())
|
if (ListJ->size() != ColFields->size())
|
||||||
PrintFatalError("Record `" + CurMap->getName() + "', field "
|
PrintFatalError("Record `" + CurMap->getName() + "', field "
|
||||||
|
|
Loading…
Reference in New Issue