[clang] [Clang] Amend SME attributes with support for ZT0. (PR #77941)
Sander de Smalen via cfe-commits
cfe-commits at lists.llvm.org
Mon Jan 15 02:49:17 PST 2024
sdesmalen-arm wrote:
> I won't approve because of the growth in FunctionProtoType — someone more qualified than me should sign off on that.
@AaronBallman or @erichkeane, would you be happy to sign off on this patch?
I think previously you raised concerns about the size of `ExtProtoInfo` when adding more bits to it. (This was discussed on https://reviews.llvm.org/D127762, but unfortunately this link currently results in a 404)
It's worth pointing out that extending `ExtProtoInfo` with some more bits doesn't change the actual size. It is 80 bytes at the moment and increasing it with 2 more bits doesn't change the total size of the struct. The size of `FunctionTypeExtraBitfields` will increase though, but I'm not sure if this is a problem because the comment above it says that these bits are uncommon to start with. I could shave off another two bits off `NumExceptionType` to avoid that, as the comment suggests 8 bits should be sufficient.
This also make me wonder if we should be reserving some extra bits for future state that may need adding?
https://github.com/llvm/llvm-project/pull/77941
More information about the cfe-commits
mailing list