[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 02/34] xen: clang: Disable built-in assembler



At 11:53 +0000 on 26 Mar (1395831232), Jan Beulich wrote:
> >>> On 25.03.14 at 17:55, <julien.grall@xxxxxxxxxx> wrote:
> > --- a/xen/Rules.mk
> > +++ b/xen/Rules.mk
> > @@ -74,6 +74,7 @@ AFLAGS-y                += -D__ASSEMBLY__ -include 
> > $(BASEDIR)/include/xen/config
> >  
> >  # Clang's built-in assembler can't handle .code16/.code32/.code64 yet
> >  AFLAGS-$(clang)         += -no-integrated-as
> > +CFLAGS-$(clang)         += -no-integrated-as
> 
> Iirc Tim had found and worked around other built-in assembler issues
> in the past, so if this is to be done unconditionally I wonder whether
> we shouldn't then drop those workarounds.

I would prefer, wherever possible, to make things work with the clang
assembler rather than rely on the binutils one forever.

BTW, I haven't looked at any of this series in detail yet but I'm
planning to go through it all tomorrow.

Cheers,

Tim.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.