[all-commits] [llvm/llvm-project] 7b96d1: [rtsan] Add dl, rt, m, and log libraries as unit t...

[Chris Apple via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 7b96d13341aa94d11c2d9ffe5a9304e3640ad80b
      https://github.com/llvm/llvm-project/commit/7b96d13341aa94d11c2d9ffe5a9304e3640ad80b
  Author: Chris Apple <cja-private at pm.me>
  Date:   2024-07-09 (Tue, 09 Jul 2024)

  Changed paths:
    M compiler-rt/lib/rtsan/tests/CMakeLists.txt

  Log Message:
  -----------
  [rtsan] Add dl, rt, m, and log libraries as unit test dependencies, get rid of atomic (#98264)

Fixing test failures in #98256 

There are two issues being fixed here:

```
ld.lld: error: undefined symbol: dlvsym
ld.lld: error: undefined symbol: dlvsym
```

Fixed by linking dl

```
FAILED: ... -lstdc++ -no-pie -pthread -latomic -m64
/opt/rh/devtoolset-11/root/usr/lib/gcc/ppc64-redhat-linux/11/../../../../bin/ld: cannot find /usr/lib64/libatomic.so.1
```

Fixed by removing atomic from the dependencies, I don't think we were
even using it.

I added in all of the dependencies used by the parent library in the
tests, hopefully getting ahead of any other issues, adding rt and m,
even if we haven't had issues with them yet



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications