[PATCH] D26083: [LV] Scalarize operands of predicated instructions

[Matthew Simpson via llvm-commits]

mssimpso updated this revision to Diff 76631.
mssimpso marked 2 inline comments as done.
mssimpso added a comment.

Addressed comments from Gil and Michael. Thanks!

- Removed unnecessary invariance check from computePredInstDiscount
- Added assert to isProfitableToScalarlize
- Added call to collectInstsToScalarize for user-selected VFs (I realized we needed this with the new assert)
- Made some auto types explicit


https://reviews.llvm.org/D26083

Files:
  lib/Transforms/Vectorize/LoopVectorize.cpp
  test/Transforms/LoopVectorize/AArch64/predication_costs.ll
  test/Transforms/LoopVectorize/if-pred-non-void.ll
  test/Transforms/LoopVectorize/if-pred-stores.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D26083.76631.patch
Type: text/x-patch
Size: 14996 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161101/3f730c79/attachment.bin>