[PATCH] D76113: [ExpandMemCmp] Correctly set alignment of generated loads

[Juneyoung Lee via Phabricator via llvm-commits]

aqjune created this revision.
Herald added subscribers: llvm-commits, danielkiss, steven.zhang, hiraditya, kristof.beyls, nemanjai.
Herald added a project: LLVM.
aqjune added a reviewer: courbet.
Herald added a subscriber: wuzish.

This is a part of the series of efforts for correcting alignment of memory operations.
(Another related bugs: https://bugs.llvm.org/show_bug.cgi?id=44388 , https://bugs.llvm.org/show_bug.cgi?id=44543 )

This fixes https://bugs.llvm.org/show_bug.cgi?id=43880 by giving default alignment of loads to 1.

The test CodeGen/AArch64/bcmp-inline-small.ll should have been changed; it was introduced by https://reviews.llvm.org/D64805 . I talked with @evandro, and confirmed that the test is okay to be changed.
Other two tests from PowerPC needed changes as well, but fixes were straightforward.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D76113

Files:
  llvm/lib/CodeGen/ExpandMemCmp.cpp
  llvm/test/CodeGen/AArch64/bcmp-inline-small.ll
  llvm/test/CodeGen/PowerPC/memCmpUsedInZeroEqualityComparison.ll
  llvm/test/CodeGen/PowerPC/memcmp-mergeexpand.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D76113.250113.patch
Type: text/x-patch
Size: 3958 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200313/c7a3cd3f/attachment.bin>