The calling ABI enum definition has been done following a similar convention in https://gcc.gnu.org/git/?p=gcc.git;a=blob;f=gcc/config/i386/i386-opts.h;h=ef2825803b32001b9632769bdff196d1e43d27ba;hb=refs/heads/master#l41
MS_ABI is used in gcc/config/i386/mingw32.h and gcc/config/i386/winnt-d.cc https://gcc.gnu.org/git/?p=gcc.git;a=blob;f=gcc/config/i386/mingw32.h;h=58304fc55f629648e47490fd3c0f3db3858e4fd8;hb=refs/heads/master#l22 These files are moved to the mingw folder in the patch series. https://gcc.gnu.org/pipermail/gcc-patches/attachments/20240221/5e75c464/attachment.txt What do you think about this change for v2? +/* Available call ABIs. */ +enum aarch64_calling_abi +{ + AARCH64_CALLING_ABI_EABI, + AARCH64_CALLING_ABI_MS, + MS_ABI = AARCH64_CALLING_ABI_MS +}; + Regards, Evgeny Thursday, February 22, 2024 12:40 PM Richard Earnshaw (lists) wrote: > +/* Available call ABIs. */ +enum calling_abi +{ + AARCH64_EABI = 0, + MS_ABI = 1 +}; + The convention in this file seems to be that all enum types to start with aarch64. Also, the enumeration values should start with the name of the enumeration type in upper case, so: enum aarch64_calling_abi { AARCH64_CALLING_ABI_EABI, AARCH64_CALLING_ABI_MS }; or something very much like that. R.