From patchwork Mon Jun 1 15:36:17 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Martin_Li=C5=A1ka?= X-Patchwork-Id: 479090 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 477881412E1 for ; Tue, 2 Jun 2015 01:36:30 +1000 (AEST) Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=gcc.gnu.org header.i=@gcc.gnu.org header.b=Lib9GAvn; dkim-atps=neutral DomainKey-Signature: a=rsa-sha1; c=nofws; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender :message-id:date:from:mime-version:to:subject:references :in-reply-to:content-type; q=dns; s=default; b=d9FVmeud8KuCaK0Na ty/RpX3Mz2z4GzFiG22HyVqpQvXlVBZfbfo4fyt6jZnk/aMuaFPF8xy1pvMSvY+z HxojdteoFTGVmp843d1M0L40002dOx+XUT51eILr+/nVHBwFK4YUXqsZ3weaNNqL VLtzfoUwDNdrEbRwcH298jbHkg= DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender :message-id:date:from:mime-version:to:subject:references :in-reply-to:content-type; s=default; bh=UIfcdke668RPhOVY22ERbyD 21Sc=; b=Lib9GAvnDE8Ar7FkdyS3fACV9vJs3P+3gRDcYrU9YmOBQTi24p9Heic pLHeG5CGno4T8BXaZpsNWnrcgHx7CA1m+JStdnEp1EFBa3MQMo3ED9vO0c+ePjaF Ekx1OGTMOUhmgNnilI41Px6JyKNPOpB2hCSXoNaYPUepWRMJmJ3I= Received: (qmail 25400 invoked by alias); 1 Jun 2015 15:36:23 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Delivered-To: mailing list gcc-patches@gcc.gnu.org Received: (qmail 25387 invoked by uid 89); 1 Jun 2015 15:36:22 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-0.5 required=5.0 tests=AWL, BAYES_00, KAM_LAZY_DOMAIN_SECURITY autolearn=no version=3.3.2 X-HELO: mx2.suse.de Received: from cantor2.suse.de (HELO mx2.suse.de) (195.135.220.15) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (CAMELLIA256-SHA encrypted) ESMTPS; Mon, 01 Jun 2015 15:36:21 +0000 Received: from relay2.suse.de (charybdis-ext.suse.de [195.135.220.254]) by mx2.suse.de (Postfix) with ESMTP id 9F581AC5F for ; Mon, 1 Jun 2015 15:36:17 +0000 (UTC) Message-ID: <556C7BF1.5010904@suse.cz> Date: Mon, 01 Jun 2015 17:36:17 +0200 From: =?UTF-8?B?TWFydGluIExpxaFrYQ==?= User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: gcc-patches@gcc.gnu.org Subject: Re: [PATCH, RFC] New memory usage statistics infrastructure References: <555604F0.40802@suse.cz> <55688F1E.4070704@arm.com> <556C444B.4040907@suse.cz> <556C6745.4010003@suse.cz> In-Reply-To: <556C6745.4010003@suse.cz> X-IsSubscribed: yes On 06/01/2015 04:08 PM, Martin Liška wrote: > On 06/01/2015 02:18 PM, Richard Biener wrote: >> On Mon, Jun 1, 2015 at 1:38 PM, Martin Liška wrote: >>> On 05/29/2015 06:09 PM, Vidya Praveen wrote: >>>> >>>> Martin, >>>> >>>> The following change: >>>> >>>> @@ -2655,10 +2655,10 @@ s-iov: build/gcov-iov$(build_exeext) $(BASEVER) $(DEVPHASE) >>>> >>>> GCOV_OBJS = gcov.o >>>> gcov$(exeext): $(GCOV_OBJS) $(LIBDEPS) >>>> - +$(LINKER) $(ALL_LINKERFLAGS) $(LDFLAGS) $(GCOV_OBJS) $(LIBS) -o $@ >>>> + +$(LINKER) $(ALL_LINKERFLAGS) $(LDFLAGS) $(GCOV_OBJS) build/hash-table.o ggc-none.o $(LIBS) -o $@ >>>> >>>> >>>> seem to cause canadian cross build failure for arm and aarch64 on x86_64 as >>>> build/hash-table.o and ggc-none.o are not built by the same compiler? >>>> >>>> arm-none-linux-gnueabi-g++ -no-pie -g -O2 -DIN_GCC -fno-exceptions -fno-rtti -fasynchronous-unwind-tables -W -Wall -Wno-narrowing >>>> +-Wwrite-strings -Wcast-qual -Wmissing-format-attribute -Woverloaded-virtual -pedantic -Wno-long-long -Wn >>>> build/hash-table.o ggc-none.o libcommon.a ../libcpp/libcpp.a ../libbacktrace/.libs/libbacktrace.a ../libiberty/libiberty.a .. >>>> +/libdecnumber/libdecnumber.a -o gcov >>>> build/hash-table.o: file not recognized: File format not recognized >>>> collect2: error: ld returned 1 exit status >>>> make[1]: *** [gcov] Error 1 >>>> >>>> >>>> Should it be: >>>> >>>> - +$(LINKER) $(ALL_LINKERFLAGS) $(LDFLAGS) $(GCOV_OBJS) $(LIBS) -o $@ >>>> + +$(LINKER) $(ALL_LINKERFLAGS) $(LDFLAGS) $(GCOV_OBJS) hash-table.o ggc-none.o $(LIBS) -o $@ >>>> >>>> instead? >>> >>> Hello Vidya. >>> >>> Thanks for pointing out. To be honest, I'm not a build system guru and it's hard for me to verify >>> that the change you suggest is the correct. >>> >>> May I please ask you for sending a patch to mailing? >> >> gcov isn't a build but a host tool so the patch looks good to me. >> >> Richard. >> >>> Thanks, >>> Martin >>> >>>> >>>> VP. >>>> >>>> >>>> On 15/05/15 15:38, Martin Liška wrote: >>>>> Hello. >>>>> >>>>> Following patch attempts to rewrite memory reports for GCC's internal allocations >>>>> so that it uses a new template type. The type shares parts which are currently duplicated, >>>>> adds support for special 'counters' and introduces new support for hash-{set,map,table}. >>>>> >>>>> Transformation of the current code is a bit tricky as we internally used hash-table as main >>>>> data structure which takes care of location-related allocations. As I want to add support even >>>>> for hash tables (and all derived types), header files inclusion and forward declaration is utilized. >>>>> >>>>> Feel free to comment the patch, as well as missing features one may want to track by location sensitive >>>>> memory allocation. >>>>> >>>>> Attachment contains sample output taken from tramp3d-v4.cpp. >>>>> >>>>> Thanks, >>>>> Martin >>>>> >>>> >>> > > Ok. > > I'm going to install following patch. > > Thanks, > Martin > Hi. Vidya notified me that I used a wrong email for his Changelog entry. This is patch I'm going to install. Thanks, Martin diff --git a/gcc/ChangeLog b/gcc/ChangeLog index daf6476..63e69d2 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,4 +1,4 @@ -2015-06-01 Vidya Praveen +2015-06-01 Vidya Praveen * Makefile.in: Fix gcov dependencies that should not point to a build folder.