This is an archive of the discontinued LLVM Phabricator instance.

[dataflow] HTMLLogger: fix off-by-one in the BB listing
ClosedPublic

Authored by sammccall on Apr 21 2023, 12:23 PM.

Details

Summary

The indexes of analysis state within a BB element is a bit odd:

BB.0 is the initial state
BB.1 is the state after the first element

etc

This means we have N+1 states and we need N+1 elements in the BB list.
We add a dummy element at the beginning rather than the end, because we want
selecting a CFG element to show the state *afterwards*.
For example, if we click on an expr, we want to be able to see its value model!

Diff Detail

Event Timeline

sammccall created this revision.Apr 21 2023, 12:23 PM
Herald added a project: Restricted Project. · View Herald Transcript
sammccall requested review of this revision.Apr 21 2023, 12:23 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 21 2023, 12:23 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript

Before:


After:

mboehme accepted this revision.Apr 24 2023, 2:04 AM
This revision is now accepted and ready to land.Apr 24 2023, 2:04 AM
This revision was landed with ongoing or failed builds.Apr 24 2023, 6:57 AM
This revision was automatically updated to reflect the committed changes.