Re: [PATCH] D13326: [PGO]: Eliminate __llvm_profile_register calls for Linux (clang changes)

2015-10-13 Thread David Li via cfe-commits
davidxl abandoned this revision. davidxl added a comment. The refactoring is done by a previous patch already. http://reviews.llvm.org/D13326 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D13326: [PGO]: Eliminate __llvm_profile_register calls for Linux (clang changes)

2015-10-06 Thread Justin Bogner via cfe-commits
David Li writes: > davidxl updated this revision to Diff 36316. > davidxl added a comment. > > I have modified the implementation to not use linker script, so this > clang patch becomes strictly refactoring with NFC. I think it is still > a good thing to have this in so that

Re: [PATCH] D13326: [PGO]: Eliminate __llvm_profile_register calls for Linux (clang changes)

2015-10-06 Thread Xinliang David Li via cfe-commits
Clang FE refactoring change is not needed anymore. In commit 0d32e7d952bc80830183e0c2c6ec5027ca6b1450, Vasileios Kalintiris did the same thing for multi-lib support. David On Tue, Oct 6, 2015 at 12:13 AM, Justin Bogner wrote: > David Li writes: >>

Re: [PATCH] D13326: [PGO]: Eliminate __llvm_profile_register calls for Linux (clang changes)

2015-10-01 Thread David Li via cfe-commits
davidxl updated this revision to Diff 36316. davidxl added a comment. I have modified the implementation to not use linker script, so this clang patch becomes strictly refactoring with NFC. I think it is still a good thing to have this in so that similar tunings like this can be done in the

[PATCH] D13326: [PGO]: Eliminate __llvm_profile_register calls for Linux (clang changes)

2015-09-30 Thread David Li via cfe-commits
davidxl created this revision. davidxl added reviewers: rsmith, bogner. davidxl added a subscriber: cfe-commits. Herald added subscribers: srhines, danalbert, tberghammer. This is the patch-1. Driver tests are also added. See http://reviews.llvm.org/D13319 for description.

Re: [PATCH] D13326: [PGO]: Eliminate __llvm_profile_register calls for Linux (clang changes)

2015-09-30 Thread David Li via cfe-commits
davidxl removed subscribers: tberghammer, danalbert, srhines. davidxl updated this revision to Diff 36186. http://reviews.llvm.org/D13326 Files: include/clang/Driver/ToolChain.h lib/Driver/SanitizerArgs.cpp lib/Driver/ToolChain.cpp lib/Driver/ToolChains.cpp lib/Driver/ToolChains.h