[PATCH] D159398: [AArch64][Clang] Disable outline atomics in freestanding env
Vladislav Khmelevsky via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Sep 4 11:29:50 PDT 2023
yota9 added a comment.
Also please keep in mind that despite of different behaviour in gcc/clang not implying outline atomics won't result in any problems, but implying them in cases where we don't want them might result in some problems.
Offtopic: Outlining atomics seems to be very CPU specific thing. In my experience LSE were ~= old exclusive semantics. So adding extra call + extra bit check (too bad IFUNCs are not used :)) each time it would be executed seems to be quite an extra load (for CPU, TLB, dcache..), so I'm not sure that outline atomics is a win-win thing (at least on some of the CPUs). This is absolutely not a case for this patch anyway, just some of my thoughts, I would be glad to hear other opinions :)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D159398/new/
https://reviews.llvm.org/D159398
More information about the cfe-commits
mailing list