aaron.ballman added a comment.

In D131255#3729304 <https://reviews.llvm.org/D131255#3729304>, @aaron.ballman 
wrote:

> After some more thought and some offline discussions, I think I have a 
> reasonable way forward: let's add `-Wsingle-bit-bitfield-constant-conversion` 
> as a new warning group under `-Wbitfield-constant-conversion` that controls 
> the diagnostic for one-bit bitfields. The diagnostic behavior here is 
> pedantically correct and will help catch some bugs for folks, but there's 
> enough existing code using `1` and not misbehaving (probably because they're 
> not inspecting the value except in a boolean context) that having separate 
> control seems useful.
>
> What do others think?

I went ahead and posted https://reviews.llvm.org/D132851.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131255

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to