On 20.03.2025 16:32, Andrew Cooper wrote: > --- a/xen/include/xen/compiler.h > +++ b/xen/include/xen/compiler.h > @@ -1,18 +1,10 @@ > #ifndef __LINUX_COMPILER_H > #define __LINUX_COMPILER_H > > -#if !defined(__GNUC__) || (__GNUC__ < 4) > -#error Sorry, your compiler is too old/not recognized. > -#elif CONFIG_CC_IS_GCC > -# if defined(CONFIG_ARM_32) && CONFIG_GCC_VERSION < 40900 > -# error Sorry, your version of GCC is too old - please use 4.9 or newer. > -# elif defined(CONFIG_ARM_64) && CONFIG_GCC_VERSION < 50100 > -/* > - * https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63293 > - * https://lore.kernel.org/r/20210107111841.gn1...@shell.armlinux.org.uk > - */ > -# error Sorry, your version of GCC is too old - please use 5.1 or newer. > -# endif > +#if CONFIG_CC_IS_GCC && CONFIG_GCC_VERSION < 50100 > +# error Sorry, your version of GCC is too old - please use 5.1 or newer > +#elif CONFIG_CC_IS_CLANG && CONFIG_CLANG_VERSION < 110000 > +# error Sorry, your version of Clang is too old - please use 11 or newer > #endif
While the Arm special cases can now indeed go away, shouldn't a RISC-V one appear instead, seeing what ./README says? Jan
