Thank you for highlighting the issue. Makes a lot of sense with same flag used for GCC IA32.

Mike

From: Marvin Häuser <mhaeuser@posteo.de>
Sent: Friday, December 16, 2022 12:08 PM
To: Pedro Falcato <pedro.falcato@gmail.com>
Cc: devel@edk2.groups.io; Feng, Bob C <bob.c.feng@intel.com>; Gao, Liming <gaoliming@byosoft.com.cn>; Chen, Christine <yuwei.chen@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>; Leif Lindholm <quic_llindhol@quicinc.com>; Andrew Fish <afish@apple.com>
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 <pedro.falcato@gmail.com> 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