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

[Krzysztof Parzyszek via llvm-commits]

kparzysz added a comment.

In http://reviews.llvm.org/D12341#234320, @hfinkel wrote:

> My understanding is that the way that llvm.expect was implemented is considered to be a mistake that we're now stuck with for backwards-compatibility reasons


One downside for using metadata is that it makes it harder to edit IR by hand.  Debugging or aliasing information is pretty much static, but branch prediction information could be manipulated by hand for experiments.  I don't know how important that is though.


http://reviews.llvm.org/D12341