JonasToth added a comment. The new switch needs documentation as well, and maybe even a note in the release notes (as it is publicly discussed as issue?). Otherwise I am fine with the changes!
================ Comment at: clang-tools-extra/test/clang-tidy/hicpp-signed-bitwise-integer-literals.cpp:19 + int Int = 30; + IResult = Int << 1; + // CHECK-MESSAGES: :[[@LINE-1]]:13: warning: use of a signed integer operand with a binary bitwise operator ---------------- Could you please add `URes << 1` as well? I believe that was problematic in the stack-overflow-question, wasn't it? Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68694/new/ https://reviews.llvm.org/D68694 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
