[PATCH] D79830: Add support of __builtin_expect_with_probability

[Erich Keane via Phabricator via cfe-commits]

erichkeane added a comment.

In D79830#2034400 <https://reviews.llvm.org/D79830#2034400>, @davidxl wrote:

> Is it possible to overload __builtin_expect(..)?


No OP, but...
First, Overloading builtins is a bit of a pain.  You end up having to do custom type checking.
Second, GCC already made the decision to do a separate name.  I'd want us to match them unless we have a really good reason not to.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D79830/new/

https://reviews.llvm.org/D79830