* [Bug other/58925] --enable-version-specific-runtime-libs breaks libcilk install
2013-10-30 12:29 [Bug other/58925] New: libtool: install: error: cannot install `libcilkrts.la' to a directory not ending in /usr/lib/gcc/x86_64-pc-linux-gnu/ octoploid at yandex dot com
@ 2013-10-30 14:59 ` octoploid at yandex dot com
2013-10-31 9:24 ` schwab@linux-m68k.org
` (6 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: octoploid at yandex dot com @ 2013-10-30 14:59 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58925
octoploid at yandex dot com changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |bviyer at gmail dot com
Summary|libtool: install: error: |--enable-version-specific-r
|cannot install |untime-libs breaks libcilk
|`libcilkrts.la' to a |install
|directory not ending in |
|/usr/lib/gcc/x86_64-pc-linu |
|x-gnu/ |
--- Comment #1 from octoploid at yandex dot com ---
$ ../gcc/configure --enable-version-specific-runtime-libs --disable-bootstrap
--disable-werror --disable-multilib --enable-languages=c,c++
is enough to reproduce the issue.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug other/58925] --enable-version-specific-runtime-libs breaks libcilk install
2013-10-30 12:29 [Bug other/58925] New: libtool: install: error: cannot install `libcilkrts.la' to a directory not ending in /usr/lib/gcc/x86_64-pc-linux-gnu/ octoploid at yandex dot com
2013-10-30 14:59 ` [Bug other/58925] --enable-version-specific-runtime-libs breaks libcilk install octoploid at yandex dot com
@ 2013-10-31 9:24 ` schwab@linux-m68k.org
2013-10-31 15:25 ` [Bug bootstrap/58925] " ebotcazou at gcc dot gnu.org
` (5 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: schwab@linux-m68k.org @ 2013-10-31 9:24 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58925
Andreas Schwab <schwab@linux-m68k.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |dimhen at gmail dot com
--- Comment #2 from Andreas Schwab <schwab@linux-m68k.org> ---
*** Bug 58935 has been marked as a duplicate of this bug. ***
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug bootstrap/58925] --enable-version-specific-runtime-libs breaks libcilk install
2013-10-30 12:29 [Bug other/58925] New: libtool: install: error: cannot install `libcilkrts.la' to a directory not ending in /usr/lib/gcc/x86_64-pc-linux-gnu/ octoploid at yandex dot com
2013-10-30 14:59 ` [Bug other/58925] --enable-version-specific-runtime-libs breaks libcilk install octoploid at yandex dot com
2013-10-31 9:24 ` schwab@linux-m68k.org
@ 2013-10-31 15:25 ` ebotcazou at gcc dot gnu.org
2013-10-31 22:02 ` octoploid at yandex dot com
` (4 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: ebotcazou at gcc dot gnu.org @ 2013-10-31 15:25 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58925
Eric Botcazou <ebotcazou at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |NEW
Last reconfirmed| |2013-10-31
CC| |ebotcazou at gcc dot gnu.org
Component|other |bootstrap
Ever confirmed|0 |1
Severity|normal |blocker
--- Comment #3 from Eric Botcazou <ebotcazou at gcc dot gnu.org> ---
I can even reproduce without --enable-version-specific-runtime-libs.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug bootstrap/58925] --enable-version-specific-runtime-libs breaks libcilk install
2013-10-30 12:29 [Bug other/58925] New: libtool: install: error: cannot install `libcilkrts.la' to a directory not ending in /usr/lib/gcc/x86_64-pc-linux-gnu/ octoploid at yandex dot com
` (2 preceding siblings ...)
2013-10-31 15:25 ` [Bug bootstrap/58925] " ebotcazou at gcc dot gnu.org
@ 2013-10-31 22:02 ` octoploid at yandex dot com
2013-10-31 22:24 ` bviyer at gmail dot com
` (3 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: octoploid at yandex dot com @ 2013-10-31 22:02 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58925
--- Comment #4 from octoploid at yandex dot com ---
The following patch fixes the issue for me:
diff --git a/libcilkrts/Makefile.am b/libcilkrts/Makefile.am
index f332cfb13de6..40a19787fda7 100644
--- a/libcilkrts/Makefile.am
+++ b/libcilkrts/Makefile.am
@@ -47,6 +47,8 @@ AM_CFLAGS = $(GENERAL_FLAGS) -std=c99
AM_CPPFLAGS = $(GENERAL_FLAGS)
AM_LDFLAGS = -lpthread -ldl
+gcc_version := $(shell cat $(top_srcdir)/../gcc/BASE-VER)
+
# Target list.
toolexeclib_LTLIBRARIES = libcilkrts.la
diff --git a/libcilkrts/Makefile.in b/libcilkrts/Makefile.in
index 35e270518211..47ea956ee457 100644
--- a/libcilkrts/Makefile.in
+++ b/libcilkrts/Makefile.in
@@ -346,6 +346,8 @@ AM_CFLAGS = $(GENERAL_FLAGS) -std=c99
AM_CPPFLAGS = $(GENERAL_FLAGS)
AM_LDFLAGS = -lpthread -ldl
+gcc_version := $(shell cat $(top_srcdir)/../gcc/BASE-VER)
+
# Target list.
toolexeclib_LTLIBRARIES = libcilkrts.la
libcilkrts_la_SOURCES = \
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug bootstrap/58925] --enable-version-specific-runtime-libs breaks libcilk install
2013-10-30 12:29 [Bug other/58925] New: libtool: install: error: cannot install `libcilkrts.la' to a directory not ending in /usr/lib/gcc/x86_64-pc-linux-gnu/ octoploid at yandex dot com
` (3 preceding siblings ...)
2013-10-31 22:02 ` octoploid at yandex dot com
@ 2013-10-31 22:24 ` bviyer at gmail dot com
2013-10-31 23:19 ` octoploid at yandex dot com
` (2 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: bviyer at gmail dot com @ 2013-10-31 22:24 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58925
--- Comment #5 from Balaji V. Iyer <bviyer at gmail dot com> ---
Hi,
I just submitted a patch to the gcc-patches mailing list. Can you try
that out?
Thanks,
Balaji V. Iyer.
On Thu, Oct 31, 2013 at 6:02 PM, octoploid at yandex dot com <
gcc-bugzilla@gcc.gnu.org> wrote:
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58925
>
> --- Comment #4 from octoploid at yandex dot com ---
> The following patch fixes the issue for me:
>
> diff --git a/libcilkrts/Makefile.am b/libcilkrts/Makefile.am
> index f332cfb13de6..40a19787fda7 100644
> --- a/libcilkrts/Makefile.am
> +++ b/libcilkrts/Makefile.am
> @@ -47,6 +47,8 @@ AM_CFLAGS = $(GENERAL_FLAGS) -std=c99
> AM_CPPFLAGS = $(GENERAL_FLAGS)
> AM_LDFLAGS = -lpthread -ldl
>
> +gcc_version := $(shell cat $(top_srcdir)/../gcc/BASE-VER)
> +
> # Target list.
> toolexeclib_LTLIBRARIES = libcilkrts.la
>
> diff --git a/libcilkrts/Makefile.in b/libcilkrts/Makefile.in
> index 35e270518211..47ea956ee457 100644
> --- a/libcilkrts/Makefile.in
> +++ b/libcilkrts/Makefile.in
> @@ -346,6 +346,8 @@ AM_CFLAGS = $(GENERAL_FLAGS) -std=c99
> AM_CPPFLAGS = $(GENERAL_FLAGS)
> AM_LDFLAGS = -lpthread -ldl
>
> +gcc_version := $(shell cat $(top_srcdir)/../gcc/BASE-VER)
> +
> # Target list.
> toolexeclib_LTLIBRARIES = libcilkrts.la
> libcilkrts_la_SOURCES = \
>
> --
> You are receiving this mail because:
> You are on the CC list for the bug.
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug bootstrap/58925] --enable-version-specific-runtime-libs breaks libcilk install
2013-10-30 12:29 [Bug other/58925] New: libtool: install: error: cannot install `libcilkrts.la' to a directory not ending in /usr/lib/gcc/x86_64-pc-linux-gnu/ octoploid at yandex dot com
` (4 preceding siblings ...)
2013-10-31 22:24 ` bviyer at gmail dot com
@ 2013-10-31 23:19 ` octoploid at yandex dot com
2013-10-31 23:55 ` bviyer at gcc dot gnu.org
2013-11-01 7:54 ` octoploid at yandex dot com
7 siblings, 0 replies; 9+ messages in thread
From: octoploid at yandex dot com @ 2013-10-31 23:19 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58925
--- Comment #6 from octoploid at yandex dot com ---
(In reply to Balaji V. Iyer from comment #5)
> Hi,
> I just submitted a patch to the gcc-patches mailing list. Can you try
> that out?
Our mails just crossed and I think your patch is fine.
Will try it out tomorrow.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug bootstrap/58925] --enable-version-specific-runtime-libs breaks libcilk install
2013-10-30 12:29 [Bug other/58925] New: libtool: install: error: cannot install `libcilkrts.la' to a directory not ending in /usr/lib/gcc/x86_64-pc-linux-gnu/ octoploid at yandex dot com
` (5 preceding siblings ...)
2013-10-31 23:19 ` octoploid at yandex dot com
@ 2013-10-31 23:55 ` bviyer at gcc dot gnu.org
2013-11-01 7:54 ` octoploid at yandex dot com
7 siblings, 0 replies; 9+ messages in thread
From: bviyer at gcc dot gnu.org @ 2013-10-31 23:55 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58925
--- Comment #7 from bviyer at gcc dot gnu.org ---
Author: bviyer
Date: Thu Oct 31 23:55:21 2013
New Revision: 204280
URL: http://gcc.gnu.org/viewcvs?rev=204280&root=gcc&view=rev
Log:
Fix for PR other/58925
+2013-10-31 Balaji V. Iyer <balaji.v.iyer@intel.com>
+
+ PR other/58925
+ * configure.ac: Added target_alias and GCC_LIBSTDCXX_RAW_CXX_FLAGS
+ fields.
+ * configure: Regenerated.
+ * aclocal.m4: Likewise.
+ * Makefile.in: Likewise.
+ * Makefile.am: Added gcc_version field.
+
Modified:
trunk/libcilkrts/ChangeLog
trunk/libcilkrts/Makefile.am
trunk/libcilkrts/Makefile.in
trunk/libcilkrts/aclocal.m4
trunk/libcilkrts/configure
trunk/libcilkrts/configure.ac
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug bootstrap/58925] --enable-version-specific-runtime-libs breaks libcilk install
2013-10-30 12:29 [Bug other/58925] New: libtool: install: error: cannot install `libcilkrts.la' to a directory not ending in /usr/lib/gcc/x86_64-pc-linux-gnu/ octoploid at yandex dot com
` (6 preceding siblings ...)
2013-10-31 23:55 ` bviyer at gcc dot gnu.org
@ 2013-11-01 7:54 ` octoploid at yandex dot com
7 siblings, 0 replies; 9+ messages in thread
From: octoploid at yandex dot com @ 2013-11-01 7:54 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58925
octoploid at yandex dot com changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED
--- Comment #8 from octoploid at yandex dot com ---
Fixed. Thanks.
BTW fib_serial() in testsuite/c-c++-common/cilk-plus/CK/fib.c is obviously
wrong:
diff --git a/gcc/testsuite/c-c++-common/cilk-plus/CK/fib.c
b/gcc/testsuite/c-c++-common/cilk-plus/CK/fib.c
index 6612936a05c0..b4aac491bf86 100644
--- a/gcc/testsuite/c-c++-common/cilk-plus/CK/fib.c
+++ b/gcc/testsuite/c-c++-common/cilk-plus/CK/fib.c
@@ -40,8 +40,8 @@ int fib_serial (int n)
return n;
else
{
- x = fib (n-1);
- y = fib (n-2);
+ x = fib_serial (n-1);
+ y = fib_serial (n-2);
return (x+y);
}
}
I've measured the serial vs. the cilk+ performance and it looks quite sad:
fib_serial:
...
fib (40) = 102334155
./a.out 1.82s user 0.00s system 99% cpu 1.822 total
fib with cilk+:
( % gcc -O3 -g -DHAVE_IO=1 -lcilkrts -ldl -fcilkplus fib.c)
...
fib (40) = 102334155
./a.out 29.14s user 0.06s system 392% cpu 7.432 total
^ permalink raw reply [flat|nested] 9+ messages in thread