aaron.ballman added inline comments.

Comment at: clang-tidy/hicpp/SignedBitwiseCheck.cpp:92
+  if (N.getNodeAs<NamedDecl>("std_type"))
+    diag(Location, "shifting a value of the standardized bitmask types");
+  else
JonasToth wrote:
> aaron.ballman wrote:
> > How about: "shifting a value of bitmask type"
> Not sure about that.
> The general bitmasks are covered by the second `diag`. 
> This one should only trigger if a shift with the standardized bitmask types 
> occurs. This exception is necessary because those are allowed for the other 
> bitwise operations(&, |, ^), but i decided that shifting them makes no sense. 
I think "standardized bitmask type" is not very clear because it's hard to know 
what is and isn't a standardized bitmask type (it's not a term of art I'm used 
to hearing, anyway). I agree that shifting by them makes no sense, but I also 
have a hard time imagining anyone is using these as shift values in the first 
place, so perhaps the diagnostic can be removed entirely unless we can find 
some code in the wild that does something like this?

  rCTE Clang Tools Extra


cfe-commits mailing list

Reply via email to