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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to