This is an archive of the discontinued LLVM Phabricator instance.

[LLDB] Add a void* cast when passing object pointers to printf %p
ClosedPublic

Authored by mstorsjo on Sep 22 2019, 12:24 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

mstorsjo created this revision.Sep 22 2019, 12:24 PM
Herald added a project: Restricted Project. · View Herald TranscriptSep 22 2019, 12:24 PM
labath accepted this revision.Sep 23 2019, 4:06 AM

This looks fine. When you see a pattern like if(log) { LLDB_LOGF(...) }, feel free to delete the surrounding if(log). It is no longer necessary, and the reason it is still there is simply because Jonas's vim script wasn't smart enough to delete it in some cases.

This revision is now accepted and ready to land.Sep 23 2019, 4:06 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptSep 23 2019, 5:03 AM