summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSimon Pilgrim <llvm-dev@redking.me.uk>2019-09-17 17:32:15 +0000
committerSimon Pilgrim <llvm-dev@redking.me.uk>2019-09-17 17:32:15 +0000
commitc191c24314cb88597429a9c2047905fe6ee7e1ed (patch)
treea371c32dfe1693148d1abe9c9bae0e74029f2cb2
parenta9a27d1ded79dd7772f2165f7939510cd8083787 (diff)
downloadbcm5719-llvm-c191c24314cb88597429a9c2047905fe6ee7e1ed.tar.gz
bcm5719-llvm-c191c24314cb88597429a9c2047905fe6ee7e1ed.zip
[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
-rw-r--r--llvm/utils/TableGen/CodeGenMapTable.cpp4
1 files changed, 2 insertions, 2 deletions
diff --git a/llvm/utils/TableGen/CodeGenMapTable.cpp b/llvm/utils/TableGen/CodeGenMapTable.cpp
index 7a5175ac354..793bb61481e 100644
--- a/llvm/utils/TableGen/CodeGenMapTable.cpp
+++ b/llvm/utils/TableGen/CodeGenMapTable.cpp
@@ -132,7 +132,7 @@ public:
MapRec->getName() + "' has empty " + "`ValueCols' field!");
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
// elements as the fields in 'ColFields'.
@@ -521,7 +521,7 @@ static void emitEnums(raw_ostream &OS, RecordKeeper &Records) {
unsigned ListSize = List->size();
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())
PrintFatalError("Record `" + CurMap->getName() + "', field "
OpenPOWER on IntegriCloud