[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