[PATCH] [MachineSink] Use the real post dominator tree

Owen Anderson resistor at mac.com
Thu Aug 7 09:24:38 PDT 2014


I definitely think we need to see compile time measurements here.  LLVM has historically avoided using algorithms based on post-dominance to avoid incurring the cost of computing it.

—Owen

On Aug 7, 2014, at 9:15 AM, Jingyue Wu <jingyue at google.com> wrote:

> Jiangning, thanks a lot for running the experiments! I should do that later as well. With that, do you think this patch is good to go? 
> 
> Just a clarification question: are these numbers compilation times or runtime performance numbers?
> 
> http://reviews.llvm.org/D4814
> 
> 
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits





More information about the llvm-commits mailing list