[PATCH] D134281: [CGP] Update MemIntrinsic alignment if possible

[Alexander Richardson via Phabricator via llvm-commits]

arichardson created this revision.
arichardson added reviewers: bcahoon, hfinkel, john.brawn, craig.topper.
Herald added subscribers: kosarev, StephenFan, kerbowa, pengfei, hiraditya, kristof.beyls, jvesely.
Herald added a project: All.
arichardson requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

Previously it was only being done if shouldAlignPointerArgs() returned
true, which right now is only true for ARM targets.
Updating the argument alignment attributes of memcpy/memset intrinsics
if the underlying object has larger alignment can be beneficial even
if CGP didn't increase alignment (as can be seen from the test changes),
so invert the loop and if condition.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D134281

Files:
  llvm/lib/CodeGen/CodeGenPrepare.cpp
  llvm/test/CodeGen/AMDGPU/lower-lds-struct-aa-memcpy.ll
  llvm/test/CodeGen/X86/mcu-abi.ll
  llvm/test/CodeGen/X86/memset-2.ll
  llvm/test/CodeGen/X86/memset64-on-x86-32.ll
  llvm/test/Transforms/CodeGenPrepare/X86/memset_chk-simplify-nobuiltin.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D134281.461555.patch
Type: text/x-patch
Size: 8451 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220920/678ef265/attachment.bin>