[llvm] Fix size in bytes of type DIEs when size in bits is not a multiple of 8 (PR #69741)

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 8 10:34:53 PST 2023


dwblaikie wrote:

I'm still a bit confused - it seems like the actual size (like what happens if you create an array of these things? I assume it's size is actually in bytes, right?) is still in bytes. But like tail padding in C++ - sometimes derived objects can use some of that padding/size for other stuff too. It doesn't mean the base class is smaller - it's got to be a whole byte for alignment reasons, etc, but the padding bits can be reused.

I think it'd be suitable to model tihs situation more like that situation - where the size is the storage size, but there's some way to describe the padding bits and allow reuse of those.

https://github.com/llvm/llvm-project/pull/69741


More information about the llvm-commits mailing list