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

[Nadav Rotem]

It is a bug in a search procedure so it does not affect the output, only the compile time. 
 
On Apr 22, 2013, at 7:46 AM, Eric Christopher <echristo at gmail.com> wrote:

> 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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130422/87ef4f27/attachment.html>