[PATCH] D12341: add llvm.unpredictable intrinsic and lower it to metadata

[Kevin B. Smith via llvm-commits]

kbsmith1 added a comment.

In http://reviews.llvm.org/D12341#235210, @kparzysz wrote:

> In http://reviews.llvm.org/D12341#235179, @kbsmith1 wrote:
>
> > [...]
>
>
> I don't think we completely understand each other.  The profiling data that is attached to MD_prof describes the behavior of the program.  You want to have something that describes the behavior of the hardware.  Those are two different things.  The former is independent from the specific properties of the hardware and can be reused when compiling for different subtargets.  The latter is not.  I'm not opposed to profiling information tied to specific hardware in principle, but I think those two kinds should be easily separable.


No, I really do get that this is for describing the behavior of the program, and I am not trying to use this to describe behavior of the HW.  All I am saying, is that you cannot simply describe the branch pattern, and expect to derive predictability information from that.  I just think that the HW feedback data can be used as a good approximation for this program behavior information, and that the branch pattern will not be usable for deriving that information.


http://reviews.llvm.org/D12341