Re: [boost] [config] BOOST_FORCEINLINE for clang

Subject: Re: [boost] [config] BOOST_FORCEINLINE for clang
From: Daryle Walker (darylew_at_[hidden])
Date: 2013-05-13 01:12:14

----------------------------------------
> Date: Sun, 12 May 2013 22:59:21 +0900
> From: mimomorin_at_[hidden]
>
> I replaced `always_inline` in boost/config/suffix.hpp with
> `__always_inline__`
> to avoid interference with a macro with the same name.
>
> Committed in r84245 ( https://svn.boost.org/trac/boost/changeset/84245 ):
> --- boost/config/suffix.hpp (revision 84244)
> +++ boost/config/suffix.hpp (working copy)
> @@ -641,7 +641,7 @@
> # if defined(_MSC_VER)
> # define BOOST_FORCEINLINE __forceinline
> # elif defined(__GNUC__) && __GNUC__ > 3
> -# define BOOST_FORCEINLINE inline __attribute__ ((always_inline))
> +# define BOOST_FORCEINLINE inline __attribute__ ((__always_inline__))
> # else
> # define BOOST_FORCEINLINE inline
> # endif

You should include a comment in the GCC section that it also works for Clang,
due to Clang's GCC emulation.  That way, later maintainers won't be wondering
if Clang support should be added.  (And help debugging if Clang ever stops
emulating GCC by default.)

Daryle W.