This is an archive of the discontinued LLVM Phabricator instance.

[dataflow] HTMLLogger: meaningful names for flow condition variables
AcceptedPublic

Authored by sammccall on Jun 21 2023, 6:10 PM.

Details

Summary

Instead of meaningless "V7", use the name "B3:1" for the flow condition
of the first analysis of basic block 3.

Diff Detail

Event Timeline

sammccall created this revision.Jun 21 2023, 6:10 PM
Herald added a project: Restricted Project. · View Herald Transcript
sammccall requested review of this revision.Jun 21 2023, 6:10 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 21 2023, 6:10 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
xazax.hun accepted this revision.Jun 23 2023, 2:09 AM

Should we have some documentation or tooltip explaining the users what the meaning of those names are?

This revision is now accepted and ready to land.Jun 23 2023, 2:09 AM
gribozavr2 accepted this revision.Jun 30 2023, 11:51 AM