Re: [PATCH] D14030: Use linker option to reference profile runtime hook (Linux)

2015-10-26 Thread Vedant Kumar via cfe-commits
vsk added a comment. This looks good to me. Does this mean we will get rid of `getInstrProfRuntimeHookVarUseFuncName`? http://reviews.llvm.org/D14030 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D14030: Use linker option to reference profile runtime hook (Linux)

2015-10-26 Thread Xinliang David Li via cfe-commits
On Mon, Oct 26, 2015 at 8:44 AM, Vedant Kumar wrote: > vsk added a comment. > > This looks good to me. Does this mean we will get rid of > `getInstrProfRuntimeHookVarUseFuncName`? > not yet -- note that this is only enabled for Linux toolchain only for now. If it works for all

[PATCH] D14030: Use linker option to reference profile runtime hook (Linux)

2015-10-23 Thread David Li via cfe-commits
davidxl created this revision. davidxl added a reviewer: bogner. davidxl added a subscriber: cfe-commits. Currently when profile instrumentation is on, the compiler emits a runtime hook use function (__llvm_profile_runtime_use) for every module that is instrumented. The generated function