This is an archive of the discontinued LLVM Phabricator instance.

[lldb] Remove most of the reproducer instrumentation
ClosedPublic

Authored by JDevlieghere on Jan 7 2022, 5:28 PM.

Details

Summary

This removes most of the reproducer instrumentation. It stills relies on the LLDB_RECORD_* macros for logging.

I'm planning a few subsequent patches:

  • Remove the LLDB_RECORD_RESULT macros
  • Rename the LLDB_RECORD_ macros to LLDB_API_LOG or something similar
  • Update lldb-instr to only generate the LLDB_API_LOG macros

Diff Detail

Event Timeline

JDevlieghere created this revision.Jan 7 2022, 5:28 PM
JDevlieghere requested review of this revision.Jan 7 2022, 5:28 PM
aprantl accepted this revision.Jan 7 2022, 5:40 PM

With regret, I accept this patch. Too bad couldn't make it work!

This revision is now accepted and ready to land.Jan 7 2022, 5:40 PM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptJan 9 2022, 9:41 PM
lldb/source/Utility/ReproducerInstrumentation.cpp