[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-commits] [qemu/qemu] 888ea9: build: Don't redefine 'inline'
From: |
GitHub |
Subject: |
[Qemu-commits] [qemu/qemu] 888ea9: build: Don't redefine 'inline' |
Date: |
Tue, 16 Feb 2016 05:30:05 -0800 |
Branch: refs/heads/master
Home: https://github.com/qemu/qemu
Commit: 888ea96aae29ce6a28afb86c2eee30068dc14d46
https://github.com/qemu/qemu/commit/888ea96aae29ce6a28afb86c2eee30068dc14d46
Author: Eric Blake <address@hidden>
Date: 2016-02-16 (Tue, 16 Feb 2016)
Changed paths:
M include/qemu/compiler.h
Log Message:
-----------
build: Don't redefine 'inline'
Actively redefining 'inline' is wrong for C++, where gcc has an
extension 'inline namespace' which fails to compile if the
keyword 'inline' is replaced by a macro expansion. This will
matter once we start to include "qemu/osdep.h" first from C++
files, depending also on whether the system headers are new
enough to be using the gcc extension.
But rather than just guard things by __cplusplus, let's look at
the overall picture. Commit df2542c737ea2 in 2007 defined 'inline'
to the gcc attribute __always_inline__, with the rationale "To
avoid discarded inlining bug". But compilers have improved since
then, and we are probably better off trusting the compiler rather
than trying to force its hand.
So just nuke our craziness.
Signed-off-by: Eric Blake <address@hidden>
Reviewed-by: Peter Maydell <address@hidden>
Message-id: address@hidden
Signed-off-by: Peter Maydell <address@hidden>
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [Qemu-commits] [qemu/qemu] 888ea9: build: Don't redefine 'inline',
GitHub <=