search

HaxeFoundation / hxcpp

Runtime files for c++ backend for haxe
Other
298 stars 190 forks source link

Atomic operations. #985

Closed Apprentice-Alchemist closed 1 year ago

Apprentice-Alchemist commented 2 years ago

See HaxeFoundation/haxe#10610

Simn commented 2 years ago

@hughsando Could you take a quick look if this is the right approach for hxcpp?

hughsando commented 2 years ago

Do these differ from the versions in include/hx/Thread.h ?

Apprentice-Alchemist commented 2 years ago

Oh right, forgot those were a thing. Let's see, there's more operations, and these only uses intrinsics (those macOS OSAtomic* functions are deprecated).

If you prefer I can merge the two sets of atomic operations.

Simn commented 2 years ago

Yes, I think that would be better.

Apprentice-Alchemist commented 2 years ago

I'm not sure why the CI hung, that test works fine on my machine.... CI works fine now.

thomasuster commented 9 months ago

Is this what broke android? https://github.com/haxenme/nme/issues/720