On 18 May 2016 at 11:56, Maxim Uvarov <maxim.uva...@linaro.org> wrote:
> On 05/18/16 18:52, Mike Holmes wrote: > >> >> >> On 18 May 2016 at 11:15, Maxim Uvarov <maxim.uva...@linaro.org <mailto: >> maxim.uva...@linaro.org>> wrote: >> >> Fix compilation error for clang with disabling 128 bit optimization. >> In function `_odp_atomic_u128_xchg_mm': >> undefined reference to `__atomic_exchange' >> >> Signed-off-by: Maxim Uvarov <maxim.uva...@linaro.org >> <mailto:maxim.uva...@linaro.org>> >> --- >> I need some quick way to make clang build happy >> >> >> Why not revert whatever introduced the issue ? >> >> . Clean patch can go later. >> >> When is "later" defined to be ? >> >> >> Why dont we just wait for the correct fix ? >> > to make -m32 work now. > why now, why do we need a fix so urgently that we dont fix it properly. > > Maxim. > >> >> >> Maxim. >> >> platform/linux-generic/include/odp_atomic_internal.h | 4 +++- >> 1 file changed, 3 insertions(+), 1 deletion(-) >> >> diff --git a/platform/linux-generic/include/odp_atomic_internal.h >> b/platform/linux-generic/include/odp_atomic_internal.h >> index 3c5606c..31c8059 100644 >> --- a/platform/linux-generic/include/odp_atomic_internal.h >> +++ b/platform/linux-generic/include/odp_atomic_internal.h >> @@ -590,7 +590,9 @@ static inline void >> _odp_atomic_flag_clear(_odp_atomic_flag_t *flag) >> /* Check if target and compiler supports 128-bit scalars and >> corresponding >> * exchange and CAS operations */ >> /* GCC on x86-64 needs -mcx16 compiler option */ >> -#if defined __SIZEOF_INT128__ && defined >> __GCC_HAVE_SYNC_COMPARE_AND_SWAP_16 >> +#if defined(__SIZEOF_INT128__) && \ >> + defined(__GCC_HAVE_SYNC_COMPARE_AND_SWAP_16) && \ >> + !defined(__clang__) >> >> /** Preprocessor symbol that indicates support for 128-bit atomics */ >> #define ODP_ATOMIC_U128 >> -- >> 2.7.1.250.gff4ea60 >> >> _______________________________________________ >> lng-odp mailing list >> lng-odp@lists.linaro.org <mailto:lng-odp@lists.linaro.org> >> https://lists.linaro.org/mailman/listinfo/lng-odp >> >> >> >> >> -- >> Mike Holmes >> Technical Manager - Linaro Networking Group >> Linaro.org <http://www.linaro.org/>***│ *Open source software for ARM >> SoCs >> "Work should be fun and collaborative, the rest follows" >> >> > -- Mike Holmes Technical Manager - Linaro Networking Group Linaro.org <http://www.linaro.org/> *│ *Open source software for ARM SoCs "Work should be fun and collaborative, the rest follows" _______________________________________________ lng-odp mailing list lng-odp@lists.linaro.org https://lists.linaro.org/mailman/listinfo/lng-odp