[PATCH] D12603: Use fixed-point representation for BranchProbability

Xinliang David Li via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 9 18:24:10 PDT 2015


I think this can be done as a follow up patch -- having
BranchProbability derive from Ratio which is derived from FixedPoint
Rep.

David

On Wed, Sep 9, 2015 at 4:04 PM, Cong Hou <congh at google.com> wrote:
> congh added a comment.
>
> In http://reviews.llvm.org/D12603#239967, @chapuni wrote:
>
>> Could we introduce fixed-point arithmetic class for general use?
>>  (It'd be not urgent)
>
>
> Sorry for the late reply! Do we need such a class in LLVM? Except BlockMass, do we have other usages or demands of fixed-point class? If yes, it is not difficult to abstract such a class from the new BranchProbability in this patch.
>
>
> http://reviews.llvm.org/D12603
>
>
>


More information about the llvm-commits mailing list