[llvm] r229059 - [unroll] Change the other worklist in the unroll analyzer to be a set

[Chandler Carruth]

On Thu, Feb 12, 2015 at 9:07 PM, Hal Finkel <hfinkel at anl.gov> wrote:

> > would visit it many times and accumulate "savings" each time.
>
> I thought the check against CountedInstructions prevented this?
>
>     if (SimpleV && CountedInstructions.insert(&I).second)
>       NumberOfOptimizedInstructions += TTI.getUserCost(&I);
>
>
>  -Hal


Ah, yes. It fixed the double count issues, just not the visitation issues.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150212/9cb7f152/attachment.html>