[PATCH] D67639: [WebAssembly] Restore defaults for stores per memop

Thomas Lively via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 16 16:15:10 PDT 2019


tlively created this revision.
tlively added reviewers: aheejin, alexcrichton.
Herald added subscribers: llvm-commits, sunfish, JDevlieghere, hiraditya, jgravelle-google, sbc100, dschuff.
Herald added a project: LLVM.

Large slowdowns were observed in Rust due to many small, constant
sized copies in conjunction with poorly-optimized memory.copy
implementations. Since memory.copy cannot be expected to be inlined
efficiently by engines at this time, stop using it for the smallest
copies. We continue to lower all memcpy intrinsics to memory.copy,
though.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D67639

Files:
  llvm/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp
  llvm/test/CodeGen/WebAssembly/bulk-memory.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D67639.220401.patch
Type: text/x-patch
Size: 4662 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190916/31d52c1f/attachment.bin>


More information about the llvm-commits mailing list