[Gc] Re: Re[8]: PATCH: Add x32 support to boehm-gc

H.J. Lu hjl.tools at gmail.com
Mon Oct 1 06:30:57 PDT 2012


On Mon, Oct 1, 2012 at 5:54 AM, Ivan Maidanski <ivmai at mail.ru> wrote:
> Hi H.J,
>
> Done both (in the same branch). Please retry.
>
> I've implemented double CAS using __sync_bool_compare_and_swap (similar to
> normal CAS on x86_64 starting from GCC 4.2). Please inspect the assembly
> code for it (as you pointed out it should be cmpxchg but not cmpxchg8b).
> Also, I don't add cast of __sync_bool_compare_and_swap result to int (I
> assume the problem should be already fixed on GCC with x32 support - please
> check that GCC does not report warnings in AO_double_compare_and_swap_full).

AO_compare_double_and_swap_double_full is wrong for x32. Why do we
need it for x32 since x32 has AO_double_compare_and_swap_full?


-- 
H.J.


More information about the Gc mailing list