[libcxx-commits] [PATCH] D97044: [libc++] [C++2b] [P0943] Add stdatomic.h header.

[Louis Dionne via Phabricator via libcxx-commits]

ldionne added a comment.

In D97044#3323396 <https://reviews.llvm.org/D97044#3323396>, @ldionne wrote:

> Looking into it. I suspect this is caused by the LLDB build doing something funky with include paths and the C standard library `<stdatomic.h>` being picked up before the libc++ one, but I'm not sure. Investigating.

I can reproduce with

  echo "#include <memory>" | clang++ -xc++ - -fmodules -fcxx-modules -std=c++11 -fsyntax-only -nostdinc++ -isystem build/default/include/c++/v1

So that's yet another modules issue. I'll revert this for the time being, however the #1 priority now needs to be to have some sort of LLDB formatters testing as part of our pre-commit CI, because we can't continue like this, it's terrible for both LLDB and libc++.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97044