[PATCH] Fix PR 23525 - Separate header mass propagation in irregular loops.

Diego Novillo dnovillo at google.com
Wed Jun 10 08:44:25 PDT 2015


On Wed, Jun 10, 2015 at 2:47 AM, David <davidxl at google.com> wrote:

> ================
> Comment at: include/llvm/Analysis/BlockFrequencyInfoImpl.h:1057
> @@ -1045,1 +1056,3 @@
> +
> +    adjustLoopHeaderMass(Loop);
>    } else {
> ----------------
> Should this be done iteratively? The mass propagation before this call is
> based on wrong initial masses for loop headers.
>

Yeah, iterating a fixed number of times here would give us a better
approximation. I was thinking of tuning this after the initial fix.

Diego.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150610/dde58f1b/attachment.html>


More information about the llvm-commits mailing list