[PATCH] D46477: [AARCH64] Change max stores for memcpy/memmov/memset and gang up loads and stores (for memcpy) for pairing.

Sirish Pande via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 14 13:24:38 PDT 2018


SirishP marked 13 inline comments as done.
SirishP added inline comments.


================
Comment at: llvm/test/CodeGen/AArch64/arm64-memset-to-bzero.ll:89
   %tmp = tail call i64 @llvm.objectsize.i64(i8* %ptr, i1 false)
-  %call = tail call i8* @__memset_chk(i8* %ptr, i32 1, i64 256, i64 %tmp)
+  %call = tail call i8* @__memset_chk(i8* %ptr, i32 1, i64 260, i64 %tmp)
   ret void
----------------
sebpop wrote:
> Do we need to change the testcase here? Why?
This was done due to change in maxstore per memset. If we change it back  to the original value, we don't need this change for now.


Repository:
  rL LLVM

https://reviews.llvm.org/D46477





More information about the llvm-commits mailing list