adams381 wrote: Addressed all review feedback in the latest push:
- Renamed isTriviallyCopyable to canPassInRegisters and isTriviallyDestructible to hasTrivialDestructor - Dropped isEmpty (can be derived from the type definition) - Dropped dataSizeInBits (equals type size, derivable from DataLayout) - Kept recordAlignInBytes — DataLayout computes wrong alignment for aligned(N) and packed SIMD structs - All three remaining fields are now printed as optional keywords (can_pass_in_registers, nontrivial_dtor, record_align(N)) and round-trip through cir-opt - Added FileCheck lit test, updated existing tests https://github.com/llvm/llvm-project/pull/188300 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
