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

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri 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:
> Instead, do
> ```
> createReplacementValues(MaybeAlign(AlignAA.getKnownAlign()), PrivatizableType.getValue(), ACS, Base, NewArgOperands);
> ```
What happens if you just do `MaybeAlign(AlignAA.getAssumedAlign())`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76550