[PATCH] D18445: [VectorUtils] Don't try and truncate PHIs to a smaller bitwidth

[James Molloy via llvm-commits]

jmolloy created this revision.
jmolloy added a reviewer: sbaranga.
jmolloy added a subscriber: llvm-commits.
jmolloy set the repository for this revision to rL LLVM.
Herald added a subscriber: mzolotukhin.

We already try not to truncate PHIs in computeMinimalBitwidths. LoopVectorize can't handle it and we really don't need to, because both induction and reduction PHIs are truncated by other means.

However, we weren't bailing out in all the places we should have, and we ended up by returning a PHI to be truncated, which has caused PR27018.

This fixes PR17018.

Repository:
  rL LLVM

http://reviews.llvm.org/D18445

Files:
  lib/Analysis/VectorUtils.cpp
  test/Transforms/LoopVectorize/AArch64/loop-vectorization-factors.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D18445.51547.patch
Type: text/x-patch
Size: 3900 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160324/cdcc9cd1/attachment.bin>