[PATCH] D95125: [WebAssembly] Enable loop unrolling

[Thomas Lively via Phabricator via llvm-commits]

tlively added a subscriber: kripken.
tlively added a comment.

cc @kripken what do you think about the size-performance trade off here?



================
Comment at: llvm/lib/Target/WebAssembly/WebAssemblyTargetTransformInfo.cpp:108
+  Loop *L, ScalarEvolution &SE, TTI::UnrollingPreferences &UP) const {
+  // Scan the loop: don't unroll loops with calls.
+  for (BasicBlock *BB : L->blocks())
----------------
Is this because loops with calls are presumed to not be hot enough to be worth unrolling?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95125