[PATCH] D35319: LSE Atomics reorg - Part I

Stefan Teleman via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 1 11:10:04 PDT 2017


On Tue, Aug 1, 2017 at 1:40 PM, Tim Northover <t.p.northover at gmail.com>
wrote:

> OK, the bug is in these lines in AArch64InstrAtomics.td:
>
>
> +  def : Pat<(!cast<SDNode>(op#"_"#size#"_acquire") GPR64sp:$Rn, SrcRHS),
> +            (!cast<Instruction>(inst#suffix) DstRHS, GPR64sp:$Rn)>;
>
> When pasting together the destination instruction you need to add the
> A/L modifiers to the non-monotonic cases. E.g.
> '!cast<Instruction>(inst # "A" # suffix)' for the acquire operation.
>
> Cheers.
>
> Tim.
>

​Yes, that was it! Thank you!​


-- 
Stefan Teleman
stefan.teleman at gmail.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170801/874ca9e1/attachment.html>


More information about the llvm-commits mailing list