bulbazord added a comment. In D146668#4221223 <https://reviews.llvm.org/D146668#4221223>, @thakis wrote:
> Hello, > https://github.com/llvm/llvm-project/commit/5fb4147f74ad90095f08f243e5f19877d09d11b9 > broke the windows build (http://45.33.8.238/win/76494/step_4.txt) because > it's now including system headers within namespace lldb. Please take a look > and revert for now if it takes a while to fix. > > (Saying that here since I see no phab link on the bad commit. Uploading > changes to phab gives you access to pre-commit testing on a bunch of > platforms, so it's useful even for post-commit reviewed patches) Thanks for letting me know and apologies for breaking the build. It looks like Benjamin Kramer fixed this in 9c6b69000d9307a36745004ee9300f70a5415160 <https://reviews.llvm.org/rG9c6b69000d9307a36745004ee9300f70a5415160>. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146668/new/ https://reviews.llvm.org/D146668 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
