[llvm] [SystemZ] Don't lower float/double ATOMIC_[LOAD|STORE] to [LOAD|STORE] (PR #75879)
Philip Reames via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 21 07:53:09 PST 2023
preames wrote:
> I don't really know what arsenm or @preames has/had in mind,
I was only getting the code back into the state it was in before I'd added complexity which never became worthwhile. I have no strong opinions on how SystemZ should or should not implement this. Having atomic loads represented by a normal load with the appropriate MMO isn't a bad idea, it just requires a lot more work through generic DAG than I had time to ever finish. If SystemZ is lowering late enough to avoid those problems (in practice..), then so be it.
https://github.com/llvm/llvm-project/pull/75879
More information about the llvm-commits
mailing list