[PATCH] D150595: [InstCombine] Simplify MemTransferInst with type inference

Nuno Lopes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 16 08:30:15 PDT 2023


nlopes added a comment.

Agreed with @nikic, this patch can't be right. The type of the alloca is not meaningful. Alas, we could replace it with a single bytes parameter in the future.
Lowering memcpys to load/store is tricky business. Let's not add more such optimizations until there's a proper solution implemented.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150595



More information about the llvm-commits mailing list