[Lldb-commits] [PATCH] D61473: ExpressionParser: only force link MCJIT when needed

2019-05-03 Thread Saleem Abdulrasool via Phabricator via lldb-commits
compnerd closed this revision. compnerd added a comment. SVN r359944 Repository: rLLDB LLDB CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61473/new/ https://reviews.llvm.org/D61473 ___ lldb-commits mailing list lldb-commits@lists.llvm.or

[Lldb-commits] [PATCH] D61473: ExpressionParser: only force link MCJIT when needed

2019-05-03 Thread Pavel Labath via Phabricator via lldb-commits
labath accepted this revision. labath added a comment. This revision is now accepted and ready to land. Thanks. Let's give that a shot. Repository: rLLDB LLDB CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61473/new/ https://reviews.llvm.org/D61473 _

[Lldb-commits] [PATCH] D61473: ExpressionParser: only force link MCJIT when needed

2019-05-03 Thread Saleem Abdulrasool via Phabricator via lldb-commits
compnerd updated this revision to Diff 198020. compnerd added a comment. Move the inclusion as suggested. Repository: rLLDB LLDB CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61473/new/ https://reviews.llvm.org/D61473 Files: source/API/SystemInitializerFull.cpp source/Plugins/Ex

[Lldb-commits] [PATCH] D61473: ExpressionParser: only force link MCJIT when needed

2019-05-03 Thread Pavel Labath via Phabricator via lldb-commits
labath requested changes to this revision. labath added a comment. This revision now requires changes to proceed. Did you run the test suite with this change? I get about 200 failures (non-shlib build), which is not surprising as without this lldb will not actually have a jit engine to evaluate

[Lldb-commits] [PATCH] D61473: ExpressionParser: only force link MCJIT when needed

2019-05-02 Thread Saleem Abdulrasool via Phabricator via lldb-commits
compnerd added a comment. gold-2.27, clang 6.0.0, `-ffunction-sections`, `-fdata-sections`, `-DLLVM_ENABLE_UNWIND_TABLES=NO`, MinSizeRel: **BEFORE** bin/lldb-server : section size addr .interp 284194928 .note.ABI-tag

[Lldb-commits] [PATCH] D61473: ExpressionParser: only force link MCJIT when needed

2019-05-02 Thread Saleem Abdulrasool via Phabricator via lldb-commits
compnerd created this revision. compnerd added reviewers: labath, clayborg, emaste, davide, xiaobai. Herald added subscribers: teemperor, abidh, krytarowski, mgorny. Herald added a project: LLDB. This was added to support FreeBSD. The inclusion of this header increases the size of `lldb-server` d