Skip to content
This repository has been archived by the owner on Mar 22, 2023. It is now read-only.

ISSUE: vcmap does not work with latest tbb (2021.1) #894

Closed
igchor opened this issue Jan 29, 2021 · 0 comments · Fixed by #935
Closed

ISSUE: vcmap does not work with latest tbb (2021.1) #894

igchor opened this issue Jan 29, 2021 · 0 comments · Fixed by #935

Comments

@igchor
Copy link
Contributor

igchor commented Jan 29, 2021

ISSUE: vcmap does not work with latest tbb (2021.1)

Environment Information

  • pmemkv version(s): 1.0/1.1/1.2/1.3/master
  • libpmemobj-cpp version(s):
  • PMDK (libpmem/libpmemobj) package version(s):
  • OS(es) version(s):
  • kernel version(s):
  • compiler, libraries, packaging and other related tools version(s):

and possibly:

  • TBB version(s): 2021.1
  • memkind version(s):
  • ndctl version(s):

Please provide a reproduction of the bug:

/home/igor/installed/tbb-2021-1.1-lock/include/tbb/../oneapi/tbb/concurrent_hash_map.h:785:31: error: call to implicitly-deleted default constructor of 'tbb::detail::d1::tbb_hash_compare<std::__cxx11::basic_string<char, std::char_traits<char>, libmemkind::pmem::allocator<char> > >'
        : concurrent_hash_map(HashCompare(), a)

How often bug is revealed:

always

Actual behavior:

Compilation fails

Expected behavior:

Compilation passes

Details

It looks like tbb specialization for std::hashstd::basic_string is not found. Maybe this has something to do with our build system - standalone program with tbb::concurrent_hash_map and std::basic_string with memkind allocator works fine.

After fixing this, we should add a CI build with latest tbb.

Additional information about Priority and Help Requested:

Are you willing to submit a pull request with a proposed change? (Yes, No)

Requested priority: (Showstopper, High, Medium, Low)

JanDorniak99 added a commit to JanDorniak99/pmemkv that referenced this issue Feb 16, 2021
JanDorniak99 added a commit to JanDorniak99/pmemkv that referenced this issue Feb 16, 2021
igchor pushed a commit to igchor/pmemkv that referenced this issue Feb 19, 2021
@karczex karczex closed this as completed Feb 22, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants