http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58996
Jeffrey A. Law law at redhat dot com changed:
What|Removed |Added
Status|NEW |RESOLVED
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58996
--- Comment #18 from bviyer at gcc dot gnu.org ---
Author: bviyer
Date: Mon Jan 20 17:49:22 2014
New Revision: 206846
URL: http://gcc.gnu.org/viewcvs?rev=206846root=gccview=rev
Log:
Fix for PR other/58996.
+2014-01-20 Balaji V. Iyer
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58996
--- Comment #14 from Barry Tannenbaum barry.m.tannenbaum at intel dot com ---
The original author of the code is no longer working on the Cilk project, so I
can't say why it's using sched_getaffinity + gettid instead the pthread
routine. But I'll
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58996
--- Comment #15 from Jakub Jelinek jakub at gcc dot gnu.org ---
The reporter of PR57298 supposedly has or had, I don't.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58996
--- Comment #16 from Balaji V. Iyer bviyer at gmail dot com ---
Hi Jakub,
Honestly, I don't know why the changes were done in aclocal.m4. I added the
change to configure.ac and then did the following commands:
autoreconf --force --verbose
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58996
--- Comment #17 from Jeffrey A. Law law at redhat dot com ---
If there's an interest, I could probably create a kickstart for a RHL8 VM
suitable for building GCC 4.9. But it's a 10+ year old platform of marginal
value at best.
I'd be much more
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58996
--- Comment #9 from Balaji V. Iyer bviyer at gmail dot com ---
Created attachment 31847
-- http://gcc.gnu.org/bugzilla/attachment.cgi?id=31847action=edit
Possible fix
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58996
--- Comment #10 from Balaji V. Iyer bviyer at gmail dot com ---
Hi Paul and Jakub,
I have attached a patch above (called Possible Fix). Does it work for
you?
Thanks,
Balaji V. Iyer.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58996
Jeffrey A. Law law at redhat dot com changed:
What|Removed |Added
Priority|P1 |P4
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58996
--- Comment #12 from Paul H. Hargrove PHHargrove at lbl dot gov ---
Balaji,
Reading the patch it looks sufficient.
I will test when I have a chance, but cannot guess how soon that may be,
-Paul
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58996
--- Comment #13 from Jakub Jelinek jakub at gcc dot gnu.org ---
Why the aclocal.m4 removals?
Also, while the rest of the HAVE_PTHREAD_AFFINITY_NP addition is one desirable
step, I've said that it is not sufficient (but of course can be done
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58996
Barry Tannenbaum barry.m.tannenbaum at intel dot com changed:
What|Removed |Added
CC|
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58996
--- Comment #7 from Jakub Jelinek jakub at gcc dot gnu.org ---
Not everything in glibc headers is meant for use by other packages, some
symbols in there are just implementation details that can change any time.
The affinity support is complicated,
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58996
--- Comment #8 from Paul H. Hargrove PHHargrove at lbl dot gov ---
(In reply to Barry Tannenbaum from comment #6)
I don't understand the reluctance to check the defintion of
__cpu_set_t_defined, since it's defined right there in
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58996
Balaji V. Iyer bviyer at gmail dot com changed:
What|Removed |Added
CC||bviyer at gmail
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58996
Jakub Jelinek jakub at gcc dot gnu.org changed:
What|Removed |Added
CC||jakub at gcc dot
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58996
--- Comment #4 from Balaji V. Iyer bviyer at gmail dot com ---
Hi Jakub,
__cpu_set_t_defined is defined in /usr/include/bits/sched.h in my SUSE
machine and in Ubuntu machine.
Thanks,
Balaji V. Iyer.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58996
--- Comment #5 from Paul H. Hargrove PHHargrove at lbl dot gov ---
The patch in comment #2 allows the build to complete without the need to
--disable-libcilkrts. However, I agree with Jakub (comment #3) that use of a
glibc internal macros is a
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58996
Richard Biener rguenth at gcc dot gnu.org changed:
What|Removed |Added
Priority|P3 |P1
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58996
Andrew Pinski pinskia at gcc dot gnu.org changed:
What|Removed |Added
Keywords||build
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58996
Richard Biener rguenth at gcc dot gnu.org changed:
What|Removed |Added
Target Milestone|--- |4.9.0
21 matches
Mail list logo