https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #38 from Terry Guo terry.guo at arm dot com ---
(In reply to Kai Tietz from comment #37)
I confirm that in libgcc we still have an issue ...
Could you please make a new report for libgcc's libgcov-util.c for it.
Thanks in advance
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #36 from Kai Tietz ktietz at gcc dot gnu.org ---
Well, I guess that you missed to reconfigure gcc. By checking current source
is the include of ftw.h guarded by HAVE_FTW_H check, which get defined by
configure if header is found.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
Terry Guo terry.guo at arm dot com changed:
What|Removed |Added
CC||terry.guo at arm dot
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #37 from Kai Tietz ktietz at gcc dot gnu.org ---
I confirm that in libgcc we still have an issue ...
Could you please make a new report for libgcc's libgcov-util.c for it.
Thanks in advance
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
Rainer Emrich rai...@emrich-ebersheim.de changed:
What|Removed |Added
Status|RESOLVED|REOPENED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #26 from Jakub Jelinek jakub at gcc dot gnu.org ---
Instead of the #undef mkdir you'd IMHO better just use (mkdir) (filename)
in the second case.
Anyway, if you've posted your patch to gcc-patches, you should be pinging it
until it is
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #30 from Kai Tietz ktietz at gcc dot gnu.org ---
Yes, this patch slipped under my radar. It would be good if you - Rainer -
would have pinged on it. As far as I recalled I awaited at that time a full
patch by Rong on this subject.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #31 from Rainer Emrich rai...@emrich-ebersheim.de ---
(In reply to Kai Tietz from comment #30)
Yes, this patch slipped under my radar. It would be good if you - Rainer -
would have pinged on it. As far as I recalled I awaited at
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #27 from Kai Tietz ktietz at gcc dot gnu.org ---
(In reply to Jakub Jelinek from comment #26)
Instead of the #undef mkdir you'd IMHO better just use (mkdir) (filename)
in the second case.
Anyway, if you've posted your patch to
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #29 from Rainer Emrich rai...@emrich-ebersheim.de ---
Am 10.02.2015 12:12, schrieb jakub at gcc dot gnu.org:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #28 from Jakub Jelinek jakub at gcc dot gnu.org ---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #28 from Jakub Jelinek jakub at gcc dot gnu.org ---
http://gcc.gnu.org/ml/gcc-patches/2014-09/msg02137.html
?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #33 from Kai Tietz ktietz at gcc dot gnu.org ---
Author: ktietz
Date: Tue Feb 10 14:14:58 2015
New Revision: 220584
URL: https://gcc.gnu.org/viewcvs?rev=220584root=gccview=rev
Log:
2015-02-10 Rainer Emrich
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
Kai Tietz ktietz at gcc dot gnu.org changed:
What|Removed |Added
Status|REOPENED|RESOLVED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #32 from Kai Tietz ktietz at gcc dot gnu.org ---
Author: ktietz
Date: Tue Feb 10 14:13:13 2015
New Revision: 220582
URL: https://gcc.gnu.org/viewcvs?rev=220582root=gccview=rev
Log:
2015-02-10 Rainer Emrich
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #23 from tbsaunde at gcc dot gnu.org ---
Author: tbsaunde
Date: Tue Feb 10 03:40:20 2015
New Revision: 220566
URL: https://gcc.gnu.org/viewcvs?rev=220566root=gccview=rev
Log:
Support gcov-tool without ftw.h
gcc/
PR
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
Jeffrey A. Law law at redhat dot com changed:
What|Removed |Added
Status|NEW |RESOLVED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
Jakub Jelinek jakub at gcc dot gnu.org changed:
What|Removed |Added
CC||jakub at gcc dot
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #21 from Kai Tietz ktietz at gcc dot gnu.org ---
(In reply to xur from comment #20)
Thanks for the comments. I'll work on this to get it fixed this time.
Let me understand your idea correctly:
We will have two patches: The first
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #11 from Rainer Emrich rai...@emrich-ebersheim.de ---
Dear friends this issue seems to become a never ending story.
In my understanding the person causing the issue is responsible for a fix.
There are several hints in this thread how
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #12 from StaffLeavers at arm dot com ---
tony.wang no longer works for ARM.
Your email will be forwarded to their line manager.
Please do not reply to this email.
If you need more information, please email real-postmas...@arm.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #13 from StaffLeavers at arm dot com ---
tony.wang no longer works for ARM.
Your email will be forwarded to their line manager.
Please do not reply to this email.
If you need more information, please email real-postmas...@arm.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
Francois-Xavier Coudert fxcoudert at gcc dot gnu.org changed:
What|Removed |Added
Status|WAITING
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #14 from StaffLeavers at arm dot com ---
tony.wang no longer works for ARM.
Your email will be forwarded to their line manager.
Please do not reply to this email.
If you need more information, please email real-postmas...@arm.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #15 from StaffLeavers at arm dot com ---
tony.wang no longer works for ARM.
Your email will be forwarded to their line manager.
Please do not reply to this email.
If you need more information, please email real-postmas...@arm.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #16 from xur at google dot com ---
I sent a patch to fix this, a few weeks ago, but I have got the review
or approval.
https://gcc.gnu.org/ml/gcc-patches/2014-09/msg00186.html
Honza, could you take a quick look?
-Rong
On Mon, Oct
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #17 from Francois-Xavier Coudert fxcoudert at gcc dot gnu.org ---
(In reply to xur from comment #16)
I sent a patch to fix this, a few weeks ago, but I have got the review
or approval.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #18 from xur at google dot com ---
This patch is after Kai Tietz's comment. and it does check the nfw headers.
On Mon, Oct 13, 2014 at 2:40 PM, fxcoudert at gcc dot gnu.org
gcc-bugzi...@gcc.gnu.org wrote:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #19 from Kai Tietz ktietz at gcc dot gnu.org ---
Hi Xur,
I asked you in my intial support to check for existance of FTW-API, and not to
implement it for Win32.
So first, send patch checking in a valid way if API can be used.
The
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #20 from xur at google dot com ---
Thanks for the comments. I'll work on this to get it fixed this time.
Let me understand your idea correctly:
We will have two patches: The first one will check FTW-API and make
the gcov-tool build
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
Francois-Xavier Coudert fxcoudert at gcc dot gnu.org changed:
What|Removed |Added
CC|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #8 from Rainer Emrich rai...@emrich-ebersheim.de ---
(In reply to xur from comment #7)
OK. I'll fix this and submit another patch.
What is the status for that?
On Wed, Aug 20, 2014 at 11:26 AM, ktietz at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #9 from Rainer Emrich rai...@emrich-ebersheim.de ---
Created attachment 33548
-- https://gcc.gnu.org/bugzilla/attachment.cgi?id=33548action=edit
Proposed patch to fix the mingw case.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
Kai Tietz ktietz at gcc dot gnu.org changed:
What|Removed |Added
Status|NEW |WAITING
Known
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #5 from xur at google dot com ---
I sent a patch for this a few days ago:
https://www.mail-archive.com/gcc-patches@gcc.gnu.org/msg87155.html
It's pending review.
Thanks,
-Rong
On Wed, Aug 20, 2014 at 10:36 AM, ktietz at gcc dot
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #6 from Kai Tietz ktietz at gcc dot gnu.org ---
Yes, I remember. I didn't comment on it.
The following checks aren't ok.
'#if !defined(_WIN32)'
you should disable those parts *only* if ftw API isn't present. This you should
check by
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
--- Comment #7 from xur at google dot com ---
OK. I'll fix this and submit another patch.
On Wed, Aug 20, 2014 at 11:26 AM, ktietz at gcc dot gnu.org
gcc-bugzi...@gcc.gnu.org wrote:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889
---
36 matches
Mail list logo