On 8 May 2018 at 16:14, Richard Henderson <[email protected]> wrote: > This implements the Atomics extension, which is mandatory for v8.1. > While testing the v8.2-SVE extension, I've run into issues with the > GCC testsuite expecting this to exist. > > Missing is the wiring up of the system registers to indicate that > the extension exists, but we have no system CPU model that would > exercise such a setting. > > Changes since v3: > * Patch 8: Do not zero-extend X[s] via the third parameter > to read_cpu_reg. > > Changes since v2: > * New patch to use a helper macro for opposite-endian > atomic_fetch_add and atomic_add_fetch, as suggested > by pm215. > * Introduce ARM_FEATURE_V8_1 and define ARM_FEATURE_V8_ATOMICS > in terms of that, reinforcing the mandatory nature of > the extension. > * Typo fix in patch 8
Applied to target-arm.next (with the atomics.h workaround first and the removal of ARM_FEATURE_V8_1), thanks. -- PMM
