[PATCH] D17460: Add prefix based function layout when profile is available.

David Li via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 20 11:04:28 PST 2016


davidxl added a comment.

Looks good in general.

1. Need a test case covering both regular case and -ffunction-sections (-function-sections for llc)

2. Need a follow up discussion to introduce user level option -freorder-function (and turn on by default with profile) in LLVM. Also talk to other maintainers of other main platforms (MachO, etc) to add this support there.


http://reviews.llvm.org/D17460





More information about the llvm-commits mailing list