On 1/24/17 8:10 PM, Saleem Abdulrasool via cfe-commits wrote:
Don't use the cast for the check, use isa. Although, since you use the value later, it is probably better to write this as: if (const auto *RD = cast<CXXRecordDecl>(CurContext)) CheckShadowInheritedVariabless(Loc, Name.getAsString(), RD, RD);
@compnerd: s/cast/dyn_cast/ or s/cast/dyn_cast_or_null/, right? Jon -- Jon Roelofs jonat...@codesourcery.com CodeSourcery / Mentor Embedded _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits