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.

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