[Openmp-commits] [PATCH] D107121: [OpenMP] Fix performance regression reported in bug #51235

Sun Aug 1 17:06:02 PDT 2021

tianshilei1992 added a comment.

In D107121#2918720 <https://reviews.llvm.org/D107121#2918720>, @protze.joachim wrote:

> Looking at your ldd output, I reallize, that we use different C++ runtime libraries. I always build against libc++. Could this make a difference?
>
> I rerun the tests with different clang compiler versions (after building libomp standalone) and can reproduce the assertion with clang 11 and clang 12.
> For depend.cpp, the assertion triggers with data = 7.

Thanks for the info. I tried with libc++, but still had no luck.

  ➜ ldd gtid
          linux-vdso.so.1 (0x00007ffd799ba000)
          libc++.so.1 => /home/shiltian/Documents/deploy/llvm/release/lib/libc++.so.1 (0x00007f05e1261000)
          libc++abi.so.1 => /home/shiltian/Documents/deploy/llvm/release/lib/libc++abi.so.1 (0x00007f05e1226000)
          libm.so.6 => /usr/lib64/libm.so.6 (0x00007f05e10e1000)
          libomp.so => /home/shiltian/Documents/deploy/openmp/release/lib/libomp.so (0x00007f05e0ffe000)
          libgcc_s.so.1 => /usr/lib64/libgcc_s.so.1 (0x00007f05e0fe3000)
          libpthread.so.0 => /usr/lib64/libpthread.so.0 (0x00007f05e0fbf000)
          libc.so.6 => /usr/lib64/libc.so.6 (0x00007f05e0df5000)
          librt.so.1 => /usr/lib64/librt.so.1 (0x00007f05e0dea000)
          libatomic.so.1 => /usr/lib64/libatomic.so.1 (0x00007f05e0de0000)
          /lib64/ld-linux-x86-64.so.2 (0x00007f05e134f000)
          libdl.so.2 => /usr/lib64/libdl.so.2 (0x00007f05e0dd9000)

Can you try with GDB to see if the extra 8 threads are created?

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D107121/new/

https://reviews.llvm.org/D107121