[PATCH] D40854: [clang-tidy] WIP implement cppcoreguidelines check for mixed integer arithmetic

Jonas Toth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 22 13:43:25 PST 2018


JonasToth added a comment.

The guideline authors will focus on mixing integer length and integer signdness.

> Thank you for the suggestion! Per our editor's discussion, we agree with "If one then mixes integer lengths and signedness in calculations confusion on what happens might occur" and would like to focus the discussion there.
>  signed and unsigned are defined in the standard and we cannot quibble with those but we can discuss mixed arithmetic. We should consider this separately.
> 
> Bjarne will add some text to this item.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D40854





More information about the cfe-commits mailing list