[llvm-commits] Cost Table, take 2

Renato Golin Linaro renato.golin at linaro.org
Tue Jan 22 03:08:30 PST 2013


Hi folks,

The logic is still the same, but I've addressed most of your concerns.

 - Moved to ADT, no more spurious includes
 - Made it a template on types and size
 - Am only creating the temporary lookup object if necessary

I wonder if the TableLookup could be used for something else, so I tried to
leave it as generic as possible and left plenty of comments (including
Evan's worry about sorting tables) for later, if proven necessary.

Please review.

cheers,
--renato
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130122/604d3080/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cost-table.patch
Type: application/octet-stream
Size: 13284 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130122/604d3080/attachment.obj>


More information about the llvm-commits mailing list