[Lldb-commits] [PATCH] D68868: Fix build under musl

2019-10-30 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere added a comment. Let me know if you want me to commit this for you. Repository: rLLDB LLDB CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68868/new/ https://reviews.llvm.org/D68868 ___ lldb-commits mailing list

[Lldb-commits] [PATCH] D68868: Fix build under musl

2019-10-14 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere accepted this revision. JDevlieghere added a comment. This revision is now accepted and ready to land. Given that the `FILE` struct lives in `stdio.h` this makes sense. Repository: rLLDB LLDB CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68868/new/

[Lldb-commits] [PATCH] D68868: Fix build under musl

2019-10-14 Thread Raphael Isemann via Phabricator via lldb-commits
teemperor added a reviewer: JDevlieghere. teemperor added a comment. (Adding Jonas because reproducers) Btw, you can just upload a diff to phabricator (e.g. the output you get with `git diff -U > fix-build.patch` when using the monorepo ) and that's

[Lldb-commits] [PATCH] D68868: Fix build under musl

2019-10-11 Thread David Demelier via Phabricator via lldb-commits
markand added a comment. In fact, in musl, the content of the structure FILE isn't exposed. But anyway, I don't understand why the code tries to write the content of the structure directly. Repository: rLLDB LLDB CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68868/new/

[Lldb-commits] [PATCH] D68868: Fix build under musl

2019-10-11 Thread David Demelier via Phabricator via lldb-commits
markand added a comment. Wait, actually the patch isn't working entirely. Repository: rLLDB LLDB CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68868/new/ https://reviews.llvm.org/D68868 ___ lldb-commits mailing list

[Lldb-commits] [PATCH] D68868: Fix build under musl

2019-10-11 Thread David Demelier via Phabricator via lldb-commits
markand created this revision. Herald added subscribers: lldb-commits, JDevlieghere, abidh. Herald added a project: LLDB. This fixes the build under musl because of missing definitions. In file included from /src/vanilla/dev/lldb/lldb-9.0.0.src/source/API/SBReproducerPrivate.h:18: