On Mon, May 9, 2022 at 10:46 AM Martin Liška <mli...@suse.cz> wrote: > > On 5/5/22 15:08, Richard Biener wrote: > > On Thu, May 5, 2022 at 2:41 PM Martin Liška <mli...@suse.cz> wrote: > >> > >> On 5/5/22 14:29, Richard Biener wrote: > >>> Can we then use static_assert (...) instead and remove the > >>> macro? > >> > >> Oh yes, we can ;) > >> > >>> Do we have C compiled code left (I think we might, > >>> otherwise we'd not have __cplusplus guards in system.h), > >>> in which case the #if should change to #ifdef __cplusplus? > >> > >> No, there's no such a consumer of the macro. > > > > OK, but for C uses it should still be different so my suggestion > > to change to #ifdef __cplusplus remains. OTOH then the change > > is somewhat pointless. > > Sure, so something like this?
Works for me. Richard. > Thanks, > Martin > > > > >> What about the updated version of the patch? > >> > >> Cheers, > >> Martin