ompt_frame_t is replaced with omp_frame_t in latest llvm-openmp library

[zygyz]

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

[zygyz]

ompt_wait_id_t --> omp_wait_id_t as well.

[jprotze]

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]

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

[SteffenSeckler]

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

[jprotze]

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.