Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/19659 )
Change subject: xcompile: silence clang when referencing packed structs' members ......................................................................
Patch Set 1:
multiple locations, and it's bogus everywhere: if we use packed structs, it's because we expect a certain memory layout (and I know, modern C compilers needn't adhere to any reasonable interpretation of that). If we refer to any such element, it
might
not be aligned, we know, we don't care.
I agree x86 and armv7 don't care (on armv7 as long as it does not cross memory map boundaries), however armv4 and mips do care. I don't know about riscv/power. That's why I wonder where clang reported this warning. Maybe it worths disabling it checking $SUBARCH not matching arm-armv4 and mips*-mips?
x86, armv7, armv8. memory map boundaries would be indicative of a larger problem given that we're looking at fields within a data structure.