joerg added inline comments.

================
Comment at: clang/lib/Driver/ToolChains/Solaris.cpp:135
     }
+    // LLVM lacks atomics support on 32-bit SPARC, so forcibly link with
+    // libatomic as a workaround.
----------------
This comment is misleading. It's not so much that LLVM doesn't support them, 
but that SPARCv8 doesn't have the necessary hardware support. The v8+ support 
is incomplete, which is a related problem though.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118021

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D118021:... Rainer Orth via Phabricator via cfe-commits
    • [PATCH] D11... John Paul Adrian Glaubitz via Phabricator via cfe-commits
    • [PATCH] D11... John Paul Adrian Glaubitz via Phabricator via cfe-commits
    • [PATCH] D11... Joerg Sonnenberger via Phabricator via cfe-commits
    • [PATCH] D11... John Paul Adrian Glaubitz via Phabricator via cfe-commits
    • [PATCH] D11... Rainer Orth via Phabricator via cfe-commits
    • [PATCH] D11... John Paul Adrian Glaubitz via Phabricator via cfe-commits
    • [PATCH] D11... Rainer Orth via Phabricator via cfe-commits
    • [PATCH] D11... John Paul Adrian Glaubitz via Phabricator via cfe-commits
    • [PATCH] D11... Rainer Orth via Phabricator via cfe-commits
    • [PATCH] D11... Rainer Orth via Phabricator via cfe-commits
    • [PATCH] D11... John Paul Adrian Glaubitz via Phabricator via cfe-commits
    • [PATCH] D11... John Paul Adrian Glaubitz via Phabricator via cfe-commits
    • [PATCH] D11... Eli Friedman via Phabricator via cfe-commits

Reply via email to