[Openmp-commits] [PATCH] D41817: [OMPT] Enable OMPT on 32-bit ARM machines

[Joachim Protze via Phabricator via Openmp-commits]

protze.joachim added a comment.

Ah, I think I understand your plan now.

A problem is, that the implementation of the gomp interface is inconsistent. Some of the functions call kmp entry points, some call the internal implementation.
In the kmp entry point function we cannot distinguish, whether we come from gomp or the application. Therefore we check for `frame==NULL` and assign the frame address if not set. This assumes that the gomp function already set the frame address if this was the way to enter the runtime.

This could actually already happen here: __builtin_frame_address(1) returns NULL in the gomp-parallel-begin-function (because the frame address could not be determined), then the value is assigned in the kmp-parallel-begin-function (this time the address can be determined, because it is within the runtime library).


Repository:
  rOMP OpenMP

https://reviews.llvm.org/D41817