[PATCH] D45098: [AArch64] fix PR32384: bump the number of stores per memset/memcpy

[Evandro Menezes via Phabricator via llvm-commits]

evandro added inline comments.


================
Comment at: llvm/test/CodeGen/AArch64/arm64-memset-to-bzero.ll:4
+; RUN: llc %s -mtriple=arm64-linux-gnu -mattr=+strict-align -o - | \
+; RUN:   FileCheck --check-prefixes=CHECK,CHECK-LINUX %s
 ; <rdar://problem/14199482> ARM64: Calls to bzero() replaced with calls to memset()
----------------
efriedma wrote:
> evandro wrote:
> > efriedma wrote:
> > > This change doesn't make any sense to me; what are you trying to do?
> > This test doesn't seem to expect that any string function is inlined, so I added `-mattr=+strict-align` to prevent this.
> Those two aren't related...?  I mean, yes, it has the right effect, but that's just a coincidence.  Please mark the functions optsize instead.
<facepalm>Of course!</facepalm>


https://reviews.llvm.org/D45098