On Wed, Mar 13, 2013 at 4:30 PM, Iyer, Balaji V <balaji.v.i...@intel.com> wrote:
> Hello Everyone,
>         This attached patch is for the Cilk Plus branch affecting the Cilk 
> Runtime. This patch should potentially allow the Cilk runtime library build 
> for non-x86 targets. We have tested this on ARM (trimslice machine) and it 
> seem to work on our test case. We encourage all the GCC target developers to 
> help us expand Cilk on other targets.

Only comment I have is:
+    x = __sync_lock_test_and_set(ptr, x);

Use the __atomic builtins instead.  This allows for better control of
what memory barriers are wanted here.

Thanks,
Andrew

Reply via email to