Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

error: no member named '_Link_type' in std::map<K,V>::iterator #58

Closed
jprotze opened this issue Mar 25, 2021 · 3 comments · Fixed by #79
Closed

error: no member named '_Link_type' in std::map<K,V>::iterator #58

jprotze opened this issue Mar 25, 2021 · 3 comments · Fixed by #79
Labels
bug Something isn't working

Comments

@jprotze
Copy link

jprotze commented Mar 25, 2021

Compiling TypeART against libc++ fails. Is _Link_type libstdc++ specific?
@ahueck ahueck added the bug Something isn't working label Mar 26, 2021
@ahueck
Copy link
Contributor

ahueck commented Mar 26, 2021

Yes, this is indeed a libstdc++ specific construct in stl_tree.h.
It is used for calculating the size of a node in the std::map impl. for our datatypes, see

TypeART/lib/runtime/AccessCountPrinter.h

Line 25 in e8655c3

sizeof(std::remove_pointer<std::map<MemAddr, PointerInfo>::iterator::_Link_type>::type) +

It makes sense to replace that with a stl-independent estimate.

@jprotze
Copy link
Author

jprotze commented Mar 26, 2021

Is this for (optional) statistics or crucial for the functionality?

@ahueck
Copy link
Contributor

ahueck commented Mar 29, 2021

Entirely optional. Just to see a very rough (max) memory usage estimate of the runtime data structures.

@ahueck ahueck mentioned this issue Apr 1, 2021
Closed
ahueck added a commit that referenced this issue Apr 5, 2021
@ahueck
remove stl dep. closes #58 and fix #60
a6d441a
@ahueck ahueck mentioned this issue Apr 14, 2021
Merged
@ahueck ahueck mentioned this issue May 5, 2021
Merged
ahueck added a commit that referenced this issue May 5, 2021
@ahueck
Merge PR #69 from feat/omp
870e4ab 
openmp support, see #10 
- Passes: Handle OpenMP codegen in LLVM IR
- Runtime: (optional) thread-safety
- Add extended testing/CI
- Test thread-safety (with sanitizers)
- Various bug fixes/improvements: #67, #65, #63, #60, #58
@ahueck ahueck mentioned this issue Jun 18, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Release 1.6 tudasc/TypeART
2 participants
@jprotze @ahueck