ahatanak added inline comments.
================
Comment at: clang/lib/Sema/SemaChecking.cpp:7788
+ if (MD && MD->isInstance())
+ ++PVIndex;
+ for (const auto *PVFormat : D->specific_attrs<FormatAttr>()) {
----------------
Isn't changing `dyn_cast<NamedDecl` to `dyn_cast<Decl>` enough to avoid the
warning? Just wondering why it is also necessary to move the above code out of
the for loop. Actually, it's not clear to me why `PVIndex` had to be
incremented in the loop in the first place.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D112569/new/
https://reviews.llvm.org/D112569
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
