[PATCH] D19299: lower __builtin_expect() directly to prof metadata instead of LLVM intrinsic

Sanjay Patel via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 20 14:44:41 PDT 2016


spatel updated this revision to Diff 54423.
spatel added a comment.

Patch updated:

1. Fixed/removed comments
2. Changed likely/unlikely profile weights to be min/max values. I'm not sure why 4 and 64 were used in LowerExpectIntrinsics, but that may not trigger the programmer's intent with builtin_expect().


http://reviews.llvm.org/D19299

Files:
  lib/CodeGen/CGBuiltin.cpp
  lib/CodeGen/CGStmt.cpp
  lib/CodeGen/CodeGenFunction.cpp
  test/CodeGen/builtin-expect.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D19299.54423.patch
Type: text/x-patch
Size: 10110 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160420/8bd76af6/attachment.bin>


More information about the cfe-commits mailing list