[Lldb-commits] [PATCH] D59056: Repair the build when LLDB_DISABLE_PYTHON is set

2019-03-06 Thread Alex Langford via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rLLDB355575: Repair the build when LLDB_DISABLE_PYTHON is set (authored by xiaobai, committed by ). Herald added a project: LLDB. Changed prior to commit: https://reviews.llvm.org/D59056?vs=189603&id=1896

[Lldb-commits] [PATCH] D59056: Repair the build when LLDB_DISABLE_PYTHON is set

2019-03-06 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. LGTM, Thank you for fixing this! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59056/new/ https://reviews.llvm.org/D59056 __

[Lldb-commits] [PATCH] D59056: Repair the build when LLDB_DISABLE_PYTHON is set

2019-03-06 Thread Alex Langford via Phabricator via lldb-commits
xiaobai updated this revision to Diff 189603. xiaobai added a comment. Group function registrations to cut down on ifdefs CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59056/new/ https://reviews.llvm.org/D59056 Files: source/API/SBReproducer.cpp Index: source/API/SBReproducer.cpp ==

[Lldb-commits] [PATCH] D59056: Repair the build when LLDB_DISABLE_PYTHON is set

2019-03-06 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere added inline comments. Comment at: source/API/SBReproducer.cpp:821 (lldb::SBTypeNameSpecifier)); +#ifndef LLDB_DISABLE_PYTHON LLDB_REGISTER_METHOD(lldb::SBTypeSummary, SBDebugger, GetSummaryForType, The order of the met

[Lldb-commits] [PATCH] D59056: Repair the build when LLDB_DISABLE_PYTHON is set

2019-03-06 Thread Alex Langford via Phabricator via lldb-commits
xiaobai created this revision. xiaobai added reviewers: JDevlieghere, labath. If LLDB_DISABLE_PYTHON is set, some functions are unavailable but SBReproducer assumes they are. Let's conditionally register those functions since they are conditionally declared. https://reviews.llvm.org/D59056 File