[clang] [llvm] [AArch64] Implement the atomic store with hint intrinsic (PR #198316)
Kerry McLaughlin via cfe-commits
cfe-commits at lists.llvm.org
Fri Oct 9 05:42:25 PDT 2026
kmclaughlin-arm wrote:
> Things to try:
>
> * Change the patch to insert the new diagnostics somewhere else in the file.
>
> * Diff the assembly to see where there are non-trivial changes.
Thank you for the suggestions, @efriedma.
I tried moving the diagnostics to the end of DiagnosticSemaKinds.td and comparing the assembly before & after, but could not find any reason for the difference still.
It looks like the way I was attempting to reproduce the compile-time regression locally was not giving accurate results. After some discussion with @c-rhodes, we started investigating the changes to AArch64InstrAtomics.td again and found that lowering the AddedComplexity value I used on the atomic hint patterns improves the compile time.
I've created a PR for this which improves the CTMark geomean by -0.12% on aarch64-O0-g: https://github.com/llvm/llvm-project/pull/230478
https://github.com/llvm/llvm-project/pull/198316
More information about the cfe-commits
mailing list