[PATCH] D73775: [clang-tidy] Cover cases like (b && c && b) in the redundant expression check

Alexey Romanov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 5 05:59:52 PST 2020


alexeyr added a comment.

In D73775#1859167 <https://reviews.llvm.org/D73775#1859167>, @aaron.ballman wrote:

> Huh, this does seem like it may be a bug in clang-tidy. I would have expected `ClangTidyDiagnosticConsumer::forwardDiagnostic()` to do this work.


>From my debugging attempt it seems not to be called (it's only supposed to be "If there is an external diagnostics engine, like in the ClangTidyPluginAction case"). Reported at https://bugs.llvm.org/show_bug.cgi?id=44799.


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

https://reviews.llvm.org/D73775





More information about the cfe-commits mailing list