[PATCH] D47587: [RISCV] Codegen support for atomic operations on RV32I

Sat Jan 15 01:29:12 PST 2022

lucascars added a comment.
Herald added subscribers: VincentWu, luke957, achieveartificialintelligence, vkmr, frasercrmck, evandro, luismarques, sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei, PkmX, the_o, brucehoult, MartinMosbeck, MaskRay, jrtc27.
Herald added a project: LLVM.

In D47587#1125544 <https://reviews.llvm.org/D47587#1125544>, @asb wrote:

> In D47587#1125518 <https://reviews.llvm.org/D47587#1125518>, @asb wrote:
>
>> Thanks for querying. I actually had been referring to an earlier draft. But even with the current draft I'm concerned that v2.0 of the ISA manual didn't specify that instr[31:28] should be ignored, meaning fence.tso may simply trap on some implementations. I'll query if anyone is tracking this behaviour across available RISC-V IP cores.
>
> Scratch that. Version 2.0 of the spec did indicate that implementations should ignore the zeroed fields, except it did so in an aside rather than the main text. I'll update to define and make use of fence.tso. Given the poor state of compliance testing in the RISC-V world there is some chance this will cause problems on some implementations despite what the spec says.

As expected... AllWinner-D1 (Risc-V 64 single core) does not implement fence.tso and will raise SIGILL...

Repository:
  rL LLVM

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

https://reviews.llvm.org/D47587