[PATCH] D102474: AArch64: support atomic load/store in GISel
Tim Northover via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri May 28 03:50:10 PDT 2021
t.p.northover added inline comments.
================
Comment at: llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp:487
// For atomics, only form anyextending loads.
if (MMO.isAtomic() && UseMI.getOpcode() != TargetOpcode::G_ANYEXT)
continue;
----------------
aemerson wrote:
> Why is this change needed? Does this line not prevent SEXTLOAD/ZEXTLOAD atomics from being formed?
Seems like it's not. I'll remove it.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D102474/new/
https://reviews.llvm.org/D102474
More information about the llvm-commits
mailing list