[PATCH] D47589: [RISCV] Add codegen support for atomic load/stores with RV32A

JF Bastien via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 31 14:03:25 PDT 2018


jfb added a comment.

In https://reviews.llvm.org/D47589#1118128, @efriedma wrote:

> The compiler-rt "__atomic_*" are known to be buggy; see https://reviews.llvm.org/D45321


Seems you're advocating that this patch (and the LangRef) follow the reality you'd like to have, not the one we actually have :-)
So I'll re-iterate: is the alignment check a documented guarantee that `__atomic_*` functions **must** provide, in all of their implementations?


https://reviews.llvm.org/D47589





More information about the llvm-commits mailing list