[testsuite] Properly set ld_library_path in cilk-plus tests

Iyer, Balaji V balaji.v.iyer@intel.com
Wed Nov 27 18:15:00 GMT 2013



> -----Original Message-----
> From: Rainer Orth [mailto:ro@CeBiTec.Uni-Bielefeld.DE]
> Sent: Wednesday, November 27, 2013 6:39 AM
> To: gcc-patches@gcc.gnu.org
> Cc: Iyer, Balaji V
> Subject: [testsuite] Properly set ld_library_path in cilk-plus tests
> 
> All 64-bit gcc.dg/atomic and c-c++-common/cilk-plus/CK execution tests
> were FAILing on Solaris 10 and 11/x86:
> 
> ld.so.1: c11-atomic-exec-1.exe: fatal: /var/gcc/regression/trunk/10-gcc-
> gas/build/./gcc/libgcc_s.so.1: wrong ELF class: ELFCLASS32
> 
> ld.so.1: fib.exe: fatal: /var/gcc/regression/trunk/10-gcc-gas/build/i386-pc-
> solaris2.10/./libcilkrts/.libs/libcilkrts.so.5: wrong ELF class: ELFCLASS32
> 
> This happens because both cilk-plus.exp files override ld_library_path
> instead of appending to it.  Fixed as follows by using the customary method
> for setting ld_library_path.  Bootstrapped without regressions on i386-pc-
> solaris2.1[01] and x86_64-unknown-linux-gnu, installed on mainline.
> 
> 	Rainer
> 
> 
> 2013-11-26  Rainer Orth  <ro@CeBiTec.Uni-Bielefeld.DE>
> 
> 	* gcc.dg/cilk-plus/cilk-plus.exp: Append to ld_library_path.
> 	Call set_ld_library_path_env_vars.
> 	* g++.dg/cilk-plus/cilk-plus.exp: Likewise.

Thanks for catching this! Sorry I didn't catch it sooner. I am just getting myself familiar with the DejaGNU framework.

-Balaji V. Iyer.



More information about the Gcc-patches mailing list