[clang] [Sema] Add check for bitfield assignments to larger integral types (PR #68276)
via cfe-commits
cfe-commits at lists.llvm.org
Sun Oct 8 02:17:37 PDT 2023
vabridgers wrote:
Hi @AaronBallman , I ran a compile using this change on clang as you asked and have results. The compile ran with no crashes or errors, and produced 1478 bitfield-conversion warnings. I'll show a few examples below. To put that number into context, I enabled -Wconversion to compare the new warning - bitfield-conversion - with the other conversion checks. You can see from the results that while 1478 findings is large, it's less than 10% of the total conversion findings that exist - so looking at those numbers in context is helpful.
Are there any other suggested improvements for this patch, or do you think it's ok to merge?
![image](https://github.com/llvm/llvm-project/assets/58314289/44313d33-6763-434b-92cb-37df8173209a)
https://github.com/llvm/llvm-project/pull/68276
More information about the cfe-commits
mailing list