[PATCH] D79830: Add support of __builtin_expect_with_probability
Erich Keane via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed May 13 10:50:20 PDT 2020
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
More information about the llvm-commits
mailing list