> On 9 Jun 2015, at 15:59, Alex Bennée <alex.ben...@linaro.org> wrote: > > > fred.kon...@greensocs.com writes: > >> From: KONRAD Frederic <fred.kon...@greensocs.com> >>
<snip> >> +DEF_HELPER_4(atomic_cmpxchg64, i32, env, i32, i64, i32) >> +DEF_HELPER_2(atomic_check, i32, env, i32) >> +DEF_HELPER_1(atomic_release, void, env) > > Apologies for being split over several mails, I didn't notice before No problem - personally prefer to keep the threads separate. > that atomic_check/release don't seem to be called at all in this patch. > Is there later work that uses it? I suspect that this is just a case of not cleaning up- I think we used check and release, and then ditched them… I’ll check (and release :-) ) Cheers Mark.