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

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 4 05:28:37 PST 2020


aaron.ballman added a comment.

In D73775#1851553 <https://reviews.llvm.org/D73775#1851553>, @alexeyr wrote:

> Also I am not sure why, but the ranges added to the diagnostic in lines 1222-1226 don't show up in the message.


Do you mean that there are no squiggly underlines under the range, or something else? Passing in a range to the diagnostics engine gives it something to put an underline under, as in what's under the -12 in: https://godbolt.org/z/GeQzYg


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

https://reviews.llvm.org/D73775





More information about the cfe-commits mailing list