[StaticAnalyzer] Silence an unused variable warning. NFC.

[george.karpenkov]

Actually, a better change would be to simply change cast<VarRegion>(MR) to VR

[mgrang]

Addressed comment.

[george.karpenkov]

LGTM, provided it compiles and tests run (ninja check-clang)

[mgrang]

In D44804#1046086, @george.karpenkov wrote:

LGTM, provided it compiles and tests run (ninja check-clang)

Thanks! Yes, I have already verified it compiles and ninja check-clang is clean.

[mgrang]

This seems to already have been fixed in r327802. Abandoning this patch.

lib/StaticAnalyzer/Checkers/RetainCountChecker.cpp

return isa<IntegerLiteral>(E) ||
          isa<CharacterLiteral>(E) ||
          isa<FloatingLiteral>(E) ||
          isa<ObjCBoolLiteralExpr>(E) ||
          isa<CXXBoolLiteralExpr>(E);
 }
 
 static Optional<std::string> describeRegion(const MemRegion *MR) {
   if (const auto *VR = dyn_cast_or_null<VarRegion>(MR))
-    return std::string(cast<VarRegion>(MR)->getDecl()->getName());
+    return std::string(VR->getDecl()->getName());
   // Once we support more storage locations for bindings,
   // this would need to be improved.
   return None;
 }
 
 /// Returns true if this stack frame is for an Objective-C method that is a
 /// property getter or setter whose body has been synthesized by the analyzer.
 static bool isSynthesizedAccessor(const StackFrameContext *SFC) {