[llvm] r179974 - SLPVectorizer: Fix a bug in the code that scans the tree in search of nodes with multiple users.

[Eric Christopher]

Testcase?

-eric

On Sun, Apr 21, 2013 at 12:37 AM, Nadav Rotem <nrotem at apple.com> wrote:
> Author: nadav
> Date: Sun Apr 21 02:37:56 2013
> New Revision: 179974
>
> URL: http://llvm.org/viewvc/llvm-project?rev=179974&view=rev
> Log:
> SLPVectorizer: Fix a bug in the code that scans the tree in search of nodes with multiple users.
> We did not terminate the switch case and we executed the search routine twice.
>
>
> Modified:
>     llvm/trunk/lib/Transforms/Vectorize/VecUtils.cpp
>
> Modified: llvm/trunk/lib/Transforms/Vectorize/VecUtils.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Vectorize/VecUtils.cpp?rev=179974&r1=179973&r2=179974&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Transforms/Vectorize/VecUtils.cpp (original)
> +++ llvm/trunk/lib/Transforms/Vectorize/VecUtils.cpp Sun Apr 21 02:37:56 2013
> @@ -354,6 +354,7 @@ void BoUpSLP::getTreeUses_rec(ArrayRef<V
>
>          getTreeUses_rec(Operands, Depth+1);
>        }
> +      return;
>      }
>      case Instruction::Store: {
>        ValueList Operands;
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits