`PaddingCalculator` Needs to re-visit all of the scalar types for padding bits:
Open
clang:codegen
clang:frontend
confirmed
- Dominant language
- LLVM
- Stars
- 40.5k
- Forks
- 18.7k
- PR merge metrics
- PR metrics pending
Description
See the comments on `getScalarOccupiedSizeInBits` here: https://github.com/llvm/llvm-project/pull/215823/
We don't seem to comprehend bool padding, nullptr_t, or enums. We should probably consider that more thoroughly before we use this for anything else.
Contributor guide
Research direction
Start with getScalarOccupiedSizeInBits and read the comments in pull request 215823. Determine the expected padding-bit behavior for bool, nullptr_t, and enum scalar types before this calculation is used elsewhere. Done means those cases are explicitly addressed with agreement on the intended behavior.
Written by the indexing model from the issue text.
Assessment
- Domain
- compilers
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Quiet
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100