apple / swift-atomics

Low-level atomic operations for Swift
Apache License 2.0
1.08k stars 51 forks source link

Simplify shims #70

Closed lorentey closed 1 year ago

lorentey commented 1 year ago

Now that we can rely on https://reviews.llvm.org/D86218, we don’t need to tiptoe around the use of _Atomic(struct _sa_dword).

Rip out the clumsy encode/decode mess and the use of __uint128_t.

As a side effect of removing the use of __uint128_t, this PR deprecates the DoubleWord.low and DoubleWord.high properties (and the associated initializer), replacing them with first and second.

Checklist

lorentey commented 1 year ago

@swift-ci test

lorentey commented 1 year ago

@swift-ci test

lorentey commented 1 year ago

@swift-ci test

lorentey commented 1 year ago

@swift-ci test

stephentyrone commented 1 year ago

deprecating high/low fills me with joy.