[llvm-commits] BlockFrequency for BasicBlocks

Jakub Staszak jstaszak at apple.com
Fri Jun 17 14:16:56 PDT 2011


The new version attached.

Thanks!
- Kuba
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bf4bb.3.patch
Type: application/octet-stream
Size: 16014 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110617/613ba110/attachment.obj>
-------------- next part --------------

On Jun 17, 2011, at 1:41 PM, Chris Lattner wrote:

> 
> On Jun 17, 2011, at 12:20 PM, Jakub Staszak wrote:
> 
>> Hi,
>> 
>> This patch introduces BlockFrequency analysis for BasicBlocks. OK to commit? 
>> Comments are welcome!
> 
> Hi Kuba,
> 
> I haven't looked at the algorithmics here, but it needs a doxygen comment on the class, should use DenseMap instead of std::map, and should prune the #includes if possible.
> 
> -Chris
> 



More information about the llvm-commits mailing list