|
From: | Richard Henderson |
Subject: | Re: [Qemu-devel] [PATCH fix for-2.9] cpus: fix wrong define name |
Date: | Mon, 10 Apr 2017 10:31:54 -0700 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 |
On 04/10/2017 10:29 AM, Peter Maydell wrote:
On 10 April 2017 at 18:26, Richard Henderson <address@hidden> wrote:On 04/09/2017 11:06 PM, Nikunj A Dadhania wrote:--- a/cpus.c +++ b/cpus.c @@ -202,7 +202,7 @@ void qemu_tcg_configure(QemuOpts *opts, Error **errp) } else if (use_icount) { error_setg(errp, "No MTTCG when icount is enabled"); } else { -#ifndef TARGET_SUPPORT_MTTCG +#ifndef TARGET_SUPPORTS_MTTCGThis sort of thing is why glibc moved to using -Wundef. It would be a huge amount of work to convert our existing sources, but it would probably pay off in the long run.We already build with -Wundef...
Ah, but then you also have to stop using #ifdef and #ifndef, and use only #if. Thus my incomplete comment above about conversion.
r~
[Prev in Thread] | Current Thread | [Next in Thread] |