[Openmp-commits] [openmp] [OpenMP][AIX]Define struct kmp_base_tas_lock with the order of two members swapped for big-endian (PR #79188)

Xing Xue via Openmp-commits openmp-commits at lists.llvm.org
Wed Jan 24 10:18:29 PST 2024


xingxue-ibm wrote:

> How did you find this? I would like to check if this affects s390x. If the repro is simple enough, maybe consider adding a test?

There were 36 `libomp/runtime` LIT test affected on AIX in 64-bit, including `critical/omp_critical_with_hint.c`, and many test cases in the `test/parallel` and `worksharing` directories. Investigating these failures shows the root cause is the memory layout of the address in 64-bit big-endian. Since s390x is big-endian, this may affect s390x as well. In terms of adding a new test case for this issue, I am not sure it will be necessary because without this patch, test cases mentioned above would be failing.


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


More information about the Openmp-commits mailing list