mboehme added inline comments.
================ Comment at: clang/unittests/Analysis/FlowSensitive/DataflowEnvironmentTest.cpp:95 - StructValue *SVal = clang::dyn_cast<StructValue>(Val); - ASSERT_NE(SVal, nullptr); - Val = SVal->getChild(*R); ---------------- Using the opportunity to simplify the code here a bit: If we `cast<>`, we don't need to assert for non-nullness because `cast<>` already does that. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154935/new/ https://reviews.llvm.org/D154935 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits