[PATCH] D76550: [Attributor] Improve the alignment of the loads

[omar ahmed via Phabricator via llvm-commits]

omarahmed added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/Attributor.cpp:5637-5640
+          unsigned Alignment =
+              AlignAA.getKnownAlign() == 0 ? 1 : AlignAA.getKnownAlign();
+          createReplacementValues(Alignment, PrivatizableType.getValue(), ACS,
+                                  Base, NewArgOperands);
----------------
lebedev.ri wrote:
> lebedev.ri wrote:
> > Instead, do
> > ```
> > createReplacementValues(MaybeAlign(AlignAA.getKnownAlign()), PrivatizableType.getValue(), ACS, Base, NewArgOperands);
> > ```
> What happens if you just do `MaybeAlign(AlignAA.getAssumedAlign())`?
some cases like argument promotion/attrs.ll returns 0 when query AAAlign attribute so that make it doesn't put any alignment to the load instr


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76550