From: Keir Fraser <keir.xen@gmail.com>
To: Tim Deegan <tim@xen.org>, xen-devel@lists.xen.org
Subject: Re: [PATCH 0 of 6] Make xen build with clang/LLVM again
Date: Thu, 05 Apr 2012 16:55:24 +0100 [thread overview]
Message-ID: <CBA37EFC.301F4%keir.xen@gmail.com> (raw)
In-Reply-To: <patchbomb.1333627633@whitby.uk.xensource.com>
On 05/04/2012 13:07, "Tim Deegan" <tim@xen.org> wrote:
> This series makes the hypervisor build with clang/LLVM again,
> after a certain amount of bit-rot.
>
> Some of these are obvious bug-fixes which are picked up by clang's
> slightly different -W* implementation. Some (the spinlock change
> in particular) are working around deficiencies in clang/llvm. I'm
> happy to change those ones to avoid any concerns people have about
> damaging the GCC build.
I don't have any comments on these beyond Jan's comments on patch 2/6. I
don't care what method you use to keep the out-of-line code ool on standard
toolchain.
-- keir
> Cheers,
>
> Tim.
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel
prev parent reply other threads:[~2012-04-05 15:55 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-04-05 12:07 [PATCH 0 of 6] Make xen build with clang/LLVM again Tim Deegan
2012-04-05 12:07 ` [PATCH 1 of 6] xen: Add -Wno-unused-value to the clang CFLAGS Tim Deegan
2012-04-05 12:07 ` [PATCH 2 of 6] x86: don't use .subsection to out-of-line failure path in spinlock.h Tim Deegan
2012-04-05 13:28 ` Jan Beulich
2012-04-05 14:05 ` Tim Deegan
2012-04-05 14:11 ` Jan Beulich
2012-04-05 12:07 ` [PATCH 3 of 6] x86/mm: Another couple of comparisons of unsigned vars with < 0 Tim Deegan
2012-04-05 13:31 ` Jan Beulich
2012-04-05 14:08 ` Tim Deegan
2012-04-05 14:19 ` Jan Beulich
2012-04-05 15:39 ` Tim Deegan
2012-04-05 12:07 ` [PATCH 4 of 6] x86: fix logical ANDs used to mask bitfields Tim Deegan
2012-04-05 13:33 ` Jan Beulich
2012-04-05 12:07 ` [PATCH 5 of 6] x86: fix memset(ptr, 0, sizeof ptr) Tim Deegan
2012-04-05 13:38 ` Jan Beulich
2012-04-05 12:07 ` [PATCH 6 of 6] x86: explicitly mark an __initdata variable as used Tim Deegan
2012-04-05 13:44 ` Jan Beulich
2012-04-05 14:11 ` Tim Deegan
2012-04-05 14:24 ` Jan Beulich
2012-04-05 15:55 ` Keir Fraser [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CBA37EFC.301F4%keir.xen@gmail.com \
--to=keir.xen@gmail.com \
--cc=tim@xen.org \
--cc=xen-devel@lists.xen.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.