[PATCH] D35319: LSE Atomics reorg - Part I

Tim Northover via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 1 09:46:47 PDT 2017


On 31 July 2017 at 17:24, Stefan Teleman via Phabricator
<reviews at reviews.llvm.org> wrote:
> Description of the problem: regardless of the atomic ordering model chosen in the test program -- in this case it's __ATOMIC_SEQ_CST -- TableGen will always match to **ldadd**, which is incorrect, instead of **ldaddal**, which is what would be expected in this case. This is shown in the lseadd-clang-O2.S assembler file indicated above.

Something is going wrong then because I can guarantee the scheme is
capable of working. Could you attach the LLVM patch that produced
these results?

Cheers.

Tim.


More information about the llvm-commits mailing list