This is an archive of the discontinued LLVM Phabricator instance.

[analyzer][NFC] Add RangeSet::dump
ClosedPublic

Authored by martong on Oct 1 2021, 1:45 AM.

Details

Summary

This tiny change improves the debugging experience of the solver a lot!

Event Timeline

martong created this revision.Oct 1 2021, 1:45 AM
martong requested review of this revision.Oct 1 2021, 1:45 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 1 2021, 1:45 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript

Please mark both of them LLVM_DUMP_METHODs. This way they will be stripped from release builds according to their documentation.

Please mark both of them LLVM_DUMP_METHODs. This way they will be stripped from release builds according to their documentation.

And Range::dump as well.

martong updated this revision to Diff 377482.Oct 6 2021, 2:51 AM
  • Rebase
  • Mark the dump methods with LLVM_DUMP_METHOD
  • Add Range::dump
This revision is now accepted and ready to land.Oct 6 2021, 9:19 AM

Thanks for the reivew!

This revision was automatically updated to reflect the committed changes.