[llvm-commits] Vectorization Factor review

[Renato Golin]

http://llvm-reviews.chandlerc.com/D335

Small patch to (hopefully) clarify what the parameters mean (width/cost)
from the std::pair<>'s first/second approach.

There's also a potentially problematic (and wrong) change by the end. I'm
assuming the cost we want to return is the cost of the cheapest width
multiplied by the width where that cost was calculated, not by the maximum
width.

It's possible that this was never a problem, if on all tests, the maximum
width is the one always returned (in which case, Width == VF).

cheers,
--renato
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130128/c8d320da/attachment.html>