[PATCH] D136822: [InstCombine] Allow memcpys from constant memory to readonly noalias parameters to be elided.

Patrick Walton via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Oct 29 14:58:22 PDT 2022


pcwalton updated this revision to Diff 471786.
pcwalton added a comment.

Addresses comments.

I actually found that this optimization was kicking in even for volatile
allocas. I assume this was a bug, so I fixed it and added a test case.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136822

Files:
  compiler-rt/test/asan/TestCases/printf-5.c
  llvm/lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp
  llvm/test/Transforms/InstCombine/memcpy-from-global.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D136822.471786.patch
Type: text/x-patch
Size: 5547 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20221029/e1c4f939/attachment.bin>


More information about the llvm-commits mailing list