This is an archive of the discontinued LLVM Phabricator instance.

Repair the build when LLDB_DISABLE_PYTHON is set
ClosedPublic

Authored by xiaobai on Mar 6 2019, 2:47 PM.

Details

Summary

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.

Diff Detail

Repository
rLLDB LLDB

Event Timeline

xiaobai created this revision.Mar 6 2019, 2:47 PM
JDevlieghere added inline comments.Mar 6 2019, 2:51 PM
source/API/SBReproducer.cpp
821

The order of the methods doesn't matter within a block, feel free to group multiple of these functions together into one ifndef.

xiaobai updated this revision to Diff 189603.Mar 6 2019, 3:07 PM

Group function registrations to cut down on ifdefs

JDevlieghere accepted this revision.Mar 6 2019, 3:24 PM

LGTM, Thank you for fixing this!

This revision is now accepted and ready to land.Mar 6 2019, 3:24 PM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptMar 6 2019, 5:02 PM