On Tue, Nov 5, 2013 at 5:17 AM, Iyer, Balaji V <balaji.v.i...@intel.com> wrote:
>
>
>> -----Original Message-----
>> From: Tom de Vries [mailto:tom_devr...@mentor.com]
>> Sent: Monday, November 4, 2013 2:15 PM
>> To: gcc@gcc.gnu.org
>> Cc: Iyer, Balaji V
>> Subject: libcilkrts breaks non-bootstrap build
>>
>> Hi,
>>
>> When configuring a gcc build with "--disable-bootstrap --enable-
>> languages=c" I run into this error:
>> ...
>> libtool: compile: g++
>> -B/home/vries/gcc_versions/devel/lean-c/install/x86_64-unknown-linux-
>> gnu/bin/
>> -B/home/vries/gcc_versions/devel/lean-c/install/x86_64-unknown-linux-
>> gnu/lib/
>> -isystem
>> /home/vries/gcc_versions/devel/lean-c/install/x86_64-unknown-linux-
>> gnu/include
>> -isystem
>> /home/vries/gcc_versions/devel/lean-c/install/x86_64-unknown-linux-
>> gnu/sys-include
>> "-DPACKAGE_NAME=\"Cilk Runtime Library\""
>> -DPACKAGE_TARNAME=\"cilk-runtime-library\" -
>> DPACKAGE_VERSION=\"2.0\"
>> "-DPACKAGE_STRING=\"Cilk Runtime Library 2.0\""
>> -DPACKAGE_BUGREPORT=\"c...@intel.com\" -DPACKAGE_URL=\"\"
>> -DPACKAGE=\"cilk-runtime-library\" -DVERSION=\"2.0\" -DSTDC_HEADERS=1
>> -DHAVE_SYS_TYPES_H=1 -DHAVE_SYS_STAT_H=1 -DHAVE_STDLIB_H=1 -
>> DHAVE_STRING_H=1
>> -DHAVE_MEMORY_H=1 -DHAVE_STRINGS_H=1 -DHAVE_INTTYPES_H=1 -
>> DHAVE_STDINT_H=1
>> -DHAVE_UNISTD_H=1 -DHAVE_ALLOCA_H=1 -DHAVE_ALLOCA=1 -
>> DHAVE_DLFCN_H=1 -DLT_OBJDIR=\".libs/\" -I. -
>> I/home/vries/gcc_versions/devel/src/libcilkrts
>> -I/home/vries/gcc_versions/devel/src/libcilkrts/include
>> -I/home/vries/gcc_versions/devel/src/libcilkrts/runtime
>> -I/home/vries/gcc_versions/devel/src/libcilkrts/runtime/config/x86
>> -DIN_CILK_RUNTIME=1 -D_Cilk_spawn= -D_Cilk_sync= -D_Cilk_for=for -
>> fcilkplus -g3
>> -O0 -dH -D_GNU_SOURCE -MT bug.lo -MD -MP -MF .deps/bug.Tpo -c
>> /home/vries/gcc_versions/devel/src/libcilkrts/runtime/bug.cpp -fPIC -DPIC -
>> o .libs/bug.o
>> g++: error: unrecognized command line option '-fcilkplus'
>> make[2]: *** [bug.lo] Error 1
>> make[2]: Leaving directory
>> `/home/vries/gcc_versions/devel/lean-c/build/x86_64-unknown-linux-
>> gnu/libcilkrts'
>> ...
>>
>> The error occurs because the compiler doesn't support -fcilkplus.
>>
>> Should configure disable libcilkplus when c++ is not enabled?
>>
>
> Is the following patch OK to fix this issue?
Ok.
Thanks,
Richard.
> Thanks,
>
> Balaji V. Iyer.
>
> Index: configure.ac
> ===================================================================
> --- configure.ac (revision 204381)
> +++ configure.ac (working copy)
> @@ -2061,7 +2061,7 @@
> case ,${enable_languages}, in
> *,c++,*) ;;
> *)
> - noconfigdirs="$noconfigdirs target-libitm target-libsanitizer
> target-libvtv"
> + noconfigdirs="$noconfigdirs target-libcilkrts target-libitm
> target-libsanitizer target-libvtv"
> ;;
> esac
>
> Index: ChangeLog
> ===================================================================
> --- ChangeLog (revision 204381)
> +++ ChangeLog (working copy)
> @@ -1,3 +1,9 @@
> +2013-11-04 Balaji V. Iyer <balaji.v.i...@intel.com>
> +
> + * configure.ac: Added libcilkrts to noconfig list when C++ is not
> + supported.
> + * configure: Regenerated.
> +
> 2013-11-01 Trevor Saunders <tsaund...@mozilla.com>
>
> * MAINTAINERS (Write After Approval): Add myself.
> Index: configure
> ===================================================================
> --- configure (revision 204381)
> +++ configure (working copy)
> @@ -6630,7 +6630,7 @@
> case ,${enable_languages}, in
> *,c++,*) ;;
> *)
> - noconfigdirs="$noconfigdirs target-libitm target-libsanitizer
> target-libvtv"
> + noconfigdirs="$noconfigdirs target-libcilkrts target-libitm
> target-libsanitizer target-libvtv"
> ;;
> esac
>
