Adding function name to the function_patch_* sections when -ffunction-sections
is provided. Helps in garbage collecting dead functions with the help of linker.
Tested:
Tested using 'make -k check-gcc RUNTESTFLAGS=i386.exp=patch*
--target_board=unix\{-m32,,-m64\}'.
2013-04-30 Harshit Chopra
2013-02-08 Harshit Chopra hars...@google.com
Porting revisions r183548, r183888, r186118, r192231, r193381.
diff --git a/gcc/c-family/c-common.c b/gcc/c-family/c-common.c
index ab416ff..04b973f 100644
--- a/gcc/c-family/c-common.c
+++ b/gcc/c-family/c-common.c
@@ -396,6 +396,13
(production binary ..)
3) Interesting examples of use cases (with graphs).
thanks,
David
On Mon, Nov 5, 2012 at 12:20 PM, Harshit Chopra hars...@google.com wrote:
Thanks David for the review. My comments are inline.
On Sat, Nov 3, 2012 at 12:38 PM, Xinliang David Li davi...@google.com
2012-11-05 Harshit Chopra hars...@google.com
* gcc/c-family/c-common.c
(handle_always_patch_for_instrumentation_attribute): Handle
always_patch_for_instrumentation attribute and turn inlining off for the
function.
(handle_never_patch_for_instrumentation_attribute): Handle
for trunk review now.
I am a bit too late now, I guess. If I target for the next release,
will it create any issues for the gcc48 release?
On Tue, Oct 30, 2012 at 5:15 PM, Harshit Chopra hars...@google.com wrote:
Adding function attributes: 'always_patch_for_instrumentation
disables inlining of that function.
Tested:
Tested by 'crosstool-validate.py --crosstool_ver=16 --testers=crosstool'
ChangeLog:
2012-10-30 Harshit Chopra hars...@google.com
* gcc/c-family/c-common.c
(handle_always_patch_for_instrumentation_attribute): Handle
commit fc3a55ccec9bc770c79f8a221f5abd397befc8f6
Author: Harshit Chopra hars...@google.com
Date: Thu Sep 20 17:49:59 2012 -0700
Instead of emitting absolute addresses to the function patch sections, emit
relative addresses. Absolute addresses might require relocation, which is time
2012-03-21 Harshit Chopra hars...@google.com
Minor changes:
i386.c: made check_should_patch_current_function C90 compatible.
i386.md: Added '\t' to bytes generated by
ix86_output_function_nops_prologue_epilogue for proper formatting
of assembly.
patch
Fixes the following tests by restricting them to 64-bit target environment.
Tested: Using 'make -k check-gcc RUNTESTFLAGS=i386.exp=patch*
--target_board=unix\{-m32,,-m64\}' and crosstool-validate.py script.
Patch to be applied to google/main branch.
2012-02-03 Harshit Chopra hars
Thanks for the review. Committed to google-main.
--
Harshit
On Fri, Feb 3, 2012 at 4:33 PM, Xinliang David Li davi...@google.com wrote:
ok for google branches.
David
On Fri, Feb 3, 2012 at 3:20 PM, Harshit Chopra hars...@google.com wrote:
Fixes the following tests by restricting them
...@redhat.com
Fabien Ch�ne fab...@gcc.gnu.org
+Harshit Chopra hars...@google.com
William Cohen wco...@redhat.com
Josh Conner jcon...@apple.com
R. Kelley Cook kc
Ping!
--
Harshit
From f63ec02c0a720174489fe450b3cc43eb00fd4bdd Mon Sep 17 00:00:00 2001
From: Harshit Chopra hars...@google.com
Date: Thu, 3 Nov 2011 17:29:23 -0700
Subject: [PATCH] Mechanism to provide efficient function instrumentation
(x86-64)
Summary: This patch aims at providing an efficient way
13 matches
Mail list logo