[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