[Lldb-commits] [lldb] [lldb][test] Turn std::atomic libcxx test generic (PR #146843)

Pavel Labath via lldb-commits lldb-commits at lists.llvm.org
Thu Jul 3 04:03:08 PDT 2025


labath wrote:

The libstdc++ test fails. Maybe we don't even have a formatter for std::atomic there? It currently looks like:
```
(std::atomic<int>) i = {
  std::__atomic_base<int> = {
    _M_i = 47
  }
}
```
.. which isn't that bad. Much better than libc++'s
```
(std::__1::atomic<int>) i = {
  std::__1::__atomic_base<int, true> = {
    std::__1::__atomic_base<int, false> = {
      __a_ = {
        std::__1::__cxx_atomic_base_impl<int> = {
          __a_value = 47
        }
      }
    }
  }
}
```
 We may still want to have a data formatter for consistency, but I suspect that's not what you're looking to do now. Maybe just move this part into the generic folder, but don't add the libstdc++ test case yet?

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


More information about the lldb-commits mailing list