On Fri Sep 29 13:13:40 2023 +0000, Martin Storsjö wrote:
Yeah, the fact that mingw headers do define `_M_<arch>` but in a header feels a bit indecisive to me; either mingw shouldn't be defining them, forcing everyone to be consistent in using the GNU style names - or they perhaps indeed should be a compiler built-in define. I somewhat disagree that clang for mingw targets should have gone with defining these though - I think that'd lead to even more confusion. If we'd want to go that way, I think we perhaps should coordinate that with GCC and make both compilers switch to doing that. In any case, I think this is a separate topic from ARM64EC :-)
I wasn't suggesting changing existing targets, I was only suggesting to not extend the problem to the new target while we still have freedom to do so.
Anyway, I will change those patches to use `__arm64ec__`.