STEllAR-GROUP / phylanx

An Asynchronous Distributed C++ Array Processing Toolkit
Boost Software License 1.0
75 stars 76 forks source link

Adding support for len() on dictionaries #1226

Closed hkaiser closed 4 years ago

hkaiser commented 4 years ago

Fixes #1224

hkaiser commented 4 years ago

@stevenrbrandt please verify whether this solve the issue you reported.

stevenrbrandt commented 4 years ago

I can't build this branch. I keep getting

[ 51%] Built target primitives.controls_dir
/usr/bin/ld: /tmp/cc3civTu.ltrans8.ltrans.o: relocation R_X86_64_PC32 against undefined symbol `_ZNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEED1Ev' can not be used when making a shared object; recompile with -fPIC
stevenrbrandt commented 4 years ago

This appears to be std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::~basic_string()

Does that mean the string library is compiled without -fPIC?

hkaiser commented 4 years ago

X86_64_PC

Which module (target) is failing to compile?

stevenrbrandt commented 4 years ago

I managed to get past the weird compilation issue. This code works for me.