[PATCH] D136281: [lld] [CMake] Link liblldCOFF against libatomic when necessary

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 3 14:33:52 PDT 2022


efriedma added a comment.

I don't think 32-bit PPC has 64-bit atomics at all, so calling libatomic is expected.  (Compare with D123872 <https://reviews.llvm.org/D123872>, where we were using libatomic when it shouldn't have been necessary.)

If the performance is important, we might want to look at changing the code to avoid 64-bit atomics. I see 64-bit std::atomic types in lld/COFF/DebugTypes.cpp, lld/MachO/ICF.cpp, and lld/include/lld/Common/Timer.h .


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D136281/new/

https://reviews.llvm.org/D136281



More information about the llvm-commits mailing list