[llvm] r194950 - [block-freq] Add BlockFrequency::scale that returns a remainder from the division and make the private scale in BlockFrequency more performant.
Michael Gottesman
mgottesman at apple.com
Sat Nov 16 20:03:10 PST 2013
The reason why I wanted to mention N/D here is I wanted to be 100% clear where the remainder was coming from. Would this be clearer?
> /// \brief Scale the given BlockFrequency by BranchProbability Prob = N/D. Return the remainder from
> + /// the division by D. Upon overflow, the routine will saturate.
Michael
On Nov 16, 2013, at 7:56 PM, Chandler Carruth <chandlerc at google.com> wrote:
>
> On Sat, Nov 16, 2013 at 7:25 PM, Michael Gottesman <mgottesman at apple.com> wrote:
> + /// \brief Scale the given BlockFrequency by N/D. Return the remainder from
> + /// the division by D. Upon overflow, the routine will saturate.
> + uint32_t scale(const BranchProbability &Prob);
> +
>
> There is no N or D here really. I would explain this more in terms of Prob?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131116/79080595/attachment.html>
More information about the llvm-commits
mailing list