Re: [PATCH] D19184: Remove MaxFunctionCount module flag annotation

2016-06-20 Thread Easwaran Raman via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL273198: Remove MaxFunctionCount module flag annotation. (authored by eraman). Changed prior to commit: http://reviews.llvm.org/D19184?vs=53969&id=61302#toc Repository: rL LLVM http://reviews.llvm.or

Re: [PATCH] D19184: Remove MaxFunctionCount module flag annotation

2016-06-20 Thread Easwaran Raman via cfe-commits
eraman added a comment. This patch and the one in http://reviews.llvm.org/D19185 are now ready to land as the inliner now uses ProfileSummaryInfo. I will check them in. http://reviews.llvm.org/D19184 ___ cfe-commits mailing list cfe-commits@lists.

Re: [PATCH] D19184: Remove MaxFunctionCount module flag annotation

2016-04-18 Thread Vedant Kumar via cfe-commits
vsk added a comment. Lgtm http://reviews.llvm.org/D19184 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D19184: Remove MaxFunctionCount module flag annotation

2016-04-15 Thread Easwaran Raman via cfe-commits
eraman created this revision. eraman added a reviewer: vsk. eraman added subscribers: cfe-commits, davidxl. Step 2 of MaxFunctionCount removal. It is superseded by ProfileSummary flag. http://reviews.llvm.org/D19184 Files: lib/CodeGen/CodeGenModule.cpp test/Profile/max-function-count.c Inde