[PATCH] blockfreq: Rewrite block frequency analysis

Duncan P. N. Exon Smith dexonsmith at apple.com
Wed Mar 19 14:01:48 PDT 2014


On Mar 19, 2014, at 1:45 PM, Chandler Carruth <chandlerc at google.com> wrote:

> 
> On Wed, Mar 19, 2014 at 1:38 PM, Duncan P. N. Exon Smith <dexonsmith at apple.com> wrote:
> Does this sound good to everyone?
> 
> Well, I already said the basic cleanup patches were good.
> 
> I've not really had a chance to look at the algorithm in detail yet because ...
>  
> 
> Chandler, do you still want something up on Phab at some point?
> 
> ... I was waiting for this before reviewing the algorithm in detail, so I still haven't had a chance to do that. If using phab is slow, I can just go through the patch files, I don't really care, it just seemed like i should wait based on prior emails.
> 

I just posted it to Phab.  However, every time I added llvm-commits at cs.uiuc.edu
as a reviewer or as a CC, it deleted it.  I also tried adding just llvm-commits
as a CC, but Phab deleted that from the field as well.

If someone call tell me what magic option I’m missing, please do!



More information about the llvm-commits mailing list