[clang] [llvm] [CIR][ABI] Add ABI metadata fields to RecordType (PR #188300)

Konstantinos Parasyris via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 7 18:55:22 PDT 2026


koparasy wrote:

> tivation is that `canPassInRegisters`, `hasTrivialDestructor`, and `recordAlignInBytes` aren't really properties of the type. They're properties of how this translation unit chose to lay the type

Thanks for adding me here. I am leaning towards @xlauko opinion on this. Mainly because:
> These three fields are TU properties, not type properties, and belong at module scope?

We do not support yet purely heterogeneous modules, and the approach I took (on incubator) was to nest them in regions, so practically both solutions would work, but conceptually these are not type properties and we should not bound them to types.  

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


More information about the cfe-commits mailing list