Thank you for highlighting the issue. Makes a lot of sense with same flag used for GCC IA32.
Mike From: Marvin Häuser <[email protected]> Sent: Friday, December 16, 2022 12:08 PM To: Pedro Falcato <[email protected]> Cc: [email protected]; Feng, Bob C <[email protected]>; Gao, Liming <[email protected]>; Chen, Christine <[email protected]>; Kinney, Michael D <[email protected]>; Leif Lindholm <[email protected]>; Andrew Fish <[email protected]> Subject: Re: [edk2-devel] [PATCH 1/1] BaseTools: Fix IA32 UINT64 alignment for CLANG toolchains Revised version of last year’s https://edk2.groups.io/g/devel/message/79670 Initially caught by last year‘s https://edk2.groups.io/g/devel/message/79318 But now caught *yet again*, independently, via bugs during development. Cc stewards so this issue finally gets some attention. Might need validation of already shipped code, who knows. Best regards, Marvin On 16. Dec 2022, at 20:54, Pedro Falcato <[email protected]<mailto:[email protected]>> wrote: PS: it's worth noting that at this moment in time, due to this, the CLANG toolchains are very broken as they break the UEFI spec and are silently incompatible with modules compiled in both GCC and MSVC. Fixing this should be top priority. Pedro -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#97516): https://edk2.groups.io/g/devel/message/97516 Mute This Topic: https://groups.io/mt/95717275/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
