[PATCH] D140089: [MemCpyOpt] Add a stack-move optimization to opportunistically merge allocas together.

Patrick Walton via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 19 16:35:08 PST 2022


pcwalton added a comment.

Should be ready for re-review now.



================
Comment at: llvm/test/Transforms/MemCpyOpt/stack-move.ll:36
+}
+
+; Tests that the optimization succeeds with a basic call to memmove.
----------------
arsenm wrote:
> Add a test with a volatile memcpy. Also one that shows metadata that was attached to these
I'm not quite sure what you meant by metadata that was attached, but I added a test `remove_alloca_metadata` that ensures that we strip metadata properly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140089



More information about the llvm-commits mailing list