[PATCH] D97667: [loop-idiom] Hoist loop memcpys to loop preheader

[Han Zhu via Phabricator via llvm-commits]

zhuhan0 updated this revision to Diff 339321.
zhuhan0 added a comment.

Fix build break. The breakage was a situation where the memcpy source and destination were of different types/sizes. Abort the transformation if that's the case. Also added a test case `memcpy-intrinsic-different-types.ll`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97667

Files:
  llvm/lib/Transforms/Scalar/LoopIdiomRecognize.cpp
  llvm/test/Transforms/LoopIdiom/memcpy-debugify-remarks.ll
  llvm/test/Transforms/LoopIdiom/memcpy-intrinsic-different-types.ll
  llvm/test/Transforms/LoopIdiom/memcpy-intrinsic.ll
  llvm/test/Transforms/LoopIdiom/memset-debugify-remarks.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D97667.339321.patch
Type: text/x-patch
Size: 31405 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210421/bb8186e6/attachment-0001.bin>