[PATCH] D131678: hicpp-signed-bitwise - Return location of the operand (and not of the operator beginning)

[Carlos Galvez via Phabricator via cfe-commits]

carlosgalvezp added a comment.

In D131678#3716213 <https://reviews.llvm.org/D131678#3716213>, @vladimir.plyashkun wrote:

> Hi @carlosgalvezp
> Yes, sorry, maybe screenshot and example from issue in our bugtracker is not the best one. 
> You can check it on this sample <https://godbolt.org/z/hxMW16vxr> from clang-tidy tests:
> The "hicpp-signed-bitwise" will point to
>
>   URes = UValue & -1;
>
> [UValue] instead of -1, which looks confusing.

Ah, yes that seems off, thanks for the clarification! :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131678