On Mon, Sep 9, 2024 at 12:21 PM Jan Beulich <jbeul...@suse.com> wrote:

> On 09.09.2024 13:03, Frediano Ziglio wrote:
> > --- a/xen/arch/x86/boot/Makefile
> > +++ b/xen/arch/x86/boot/Makefile
> > @@ -15,7 +15,7 @@ CFLAGS_x86_32 := $(subst -m64,-m32
> -march=i686,$(XEN_TREEWIDE_CFLAGS))
> >  $(call cc-options-add,CFLAGS_x86_32,CC,$(EMBEDDED_EXTRA_CFLAGS))
> >  CFLAGS_x86_32 += -Werror -fno-builtin -g0 -msoft-float -mregparm=3
> >  CFLAGS_x86_32 += -nostdinc -include $(filter
> %/include/xen/config.h,$(XEN_CFLAGS))
> > -CFLAGS_x86_32 += $(filter -I% -O%,$(XEN_CFLAGS))
> > +CFLAGS_x86_32 += $(filter -I% -O%,$(XEN_CFLAGS)) -D__XEN__
>
> Any reason not to similarly make this -D% as a filter expression?
>
> Jan
>

I saw that a lot of defines are about processor support which do not make
sense for 32 bit.

Do you want me to add a comment in the commit?

Frediano

Reply via email to