On 06/01/2015 02:18 PM, Richard Biener wrote:
> On Mon, Jun 1, 2015 at 1:38 PM, Martin Liška <mli...@suse.cz> 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
>From 04939698092c834713bd83d49bc50a5ab31dbe79 Mon Sep 17 00:00:00 2001
From: mliska <mli...@suse.cz>
Date: Mon, 1 Jun 2015 16:03:30 +0200
Subject: [PATCH] Fix Makefile dependencies.

gcc/ChangeLog:

2015-06-01  Vidya Praveen  <vidya.prav...@atmel.com>
	* Makefile.in: Fix gcov dependencies that should
	not point to a build folder.
---
 gcc/Makefile.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gcc/Makefile.in b/gcc/Makefile.in
index b59b5d9..952f285 100644
--- a/gcc/Makefile.in
+++ b/gcc/Makefile.in
@@ -2667,7 +2667,7 @@ s-iov: build/gcov-iov$(build_exeext) $(BASEVER) $(DEVPHASE)
 GCOV_OBJS = gcov.o
 gcov$(exeext): $(GCOV_OBJS) $(LIBDEPS)
 	+$(LINKER) $(ALL_LINKERFLAGS) $(LDFLAGS) $(GCOV_OBJS) \
-		build/hash-table.o ggc-none.o $(LIBS) -o $@
+		hash-table.o ggc-none.o $(LIBS) -o $@
 GCOV_DUMP_OBJS = gcov-dump.o
 gcov-dump$(exeext): $(GCOV_DUMP_OBJS) $(LIBDEPS)
 	+$(LINKER) $(ALL_LINKERFLAGS) $(LDFLAGS) $(GCOV_DUMP_OBJS) \
-- 
2.1.4

Reply via email to