[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