search

PRUNERS / archer

Archer, a data race detection tool for large OpenMP applications
https://pruners.github.io/archer
Apache License 2.0
61 stars 13 forks source link

ompt_frame_t is replaced with omp_frame_t in latest llvm-openmp library #86

Open zygyz opened 5 years ago

zygyz commented 5 years ago

Compilation error is generated in ompt-tsan.cpp. 'ompt_frame_t' is no longer used and is replaced with omp_frame_t.

zygyz commented 5 years ago

ompt_wait_id_t --> omp_wait_id_t as well.

jprotze commented 5 years ago

If there are issues, currently please use the release_60 tag of the OpenMP runtime, or just the release 6.0 binaries. Once release_70 will be out, we will update to that interface.

I hope, with LLVM 8.0 release the OMPT interface will be stable and we also try to deliver the archer runtime with LLVM 8.0.

SteffenSeckler commented 5 years ago

any update on this? even https://reviews.llvm.org/D45890 still doesn't look to be updated accordingly

SteffenSeckler commented 5 years ago

or is the solution to use https://github.com/PRUNERS/openmp/tree/archer_80_reduction ?

jprotze commented 5 years ago

The final OpenMP 5.0 specification says ompt_frame_t and ompt_wait_id_t. Not that we would really use the frame_t in Archer. This is also the typedef delivered with llvm/8.0.

Both https://reviews.llvm.org/D45890 and https://github.com/PRUNERS/openmp/tree/archer_80_reduction are right. The latter branch also contains an implementation of the reduction event, we did not push to upstream yet.