[PATCH] D134323: AtomicExpand: Avoid some operations if the atomic is overaligned

James Y Knight via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 22 07:58:05 PDT 2022


jyknight added inline comments.


================
Comment at: llvm/lib/CodeGen/AtomicExpandPass.cpp:701
 
-  PMV.AlignedAddrAlignment = Align(MinWordSize);
+  if (AddrAlign < MinWordSize) {
+    IntegerType *IntTy = DL.getIntPtrType(Ctx, PtrTy->getAddressSpace());
----------------
Shouldn't this folding get handled by the IR folder already? Why do we need to special case it here?


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

https://reviews.llvm.org/D134323



More information about the llvm-commits mailing list