[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