[PATCH] D131318: [clang-tidy] Add llvm-derefencing-dyn-cast check

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Aug 6 03:33:39 PDT 2022


njames93 added a comment.

I had thought about merging this into the prefer-isa-or-dyn-cast in conditionals check, however there just isn't enough cross-over between the 2 and the names wouldn't really match up with what each part of the check does.
There could be merit in merging them both into a new check and removing the old one, however I feel that could cause disruption for downstream users.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D131318/new/

https://reviews.llvm.org/D131318



More information about the cfe-commits mailing list