[Openmp-commits] [openmp] [OpenMP][AArch64] Workaround for ompt/synchronization tests (PR #75848)

Carlos Eduardo Seo via Openmp-commits openmp-commits at lists.llvm.org
Tue Dec 19 06:05:20 PST 2023


ceseo wrote:

> The OMPT tests test the codeptr address for plausibility. The main goal is to test that the codeptr does not point to a completely other part of the code (e.g. a frame nested in the runtime call). Adding another value to the `print_possible_return_addresses` function means that the test will accept an additional value for codeptr. This will not break tests but potentially unbreak tests. We don't care too much about the precise value at this point.
> 
> For a precise testing, we would need to involve debug information and check whether we can successfully map back the codeptr address to the source code line. At the same time, this approach would allow to test the quality of debug information generated by the compiler. The main reason for not implementing such testing yet is that it seemed cumbersome to set up a portable workflow considering different `addr2line` tools, supported dwarf versions by the installed tools, _locales_ settings ... .

Right, so I think it makes sense to just drop the OS check for now. I'll re-sbumit the PR.

https://github.com/llvm/llvm-project/pull/75848


More information about the Openmp-commits mailing list