This is an archive of the discontinued LLVM Phabricator instance.

[NFC] Fix comment describing what EdgeBundles is
ClosedPublic

Authored by mkazantsev on Mar 22 2021, 10:23 PM.

Details

Summary

The original comment says the same thing twice, and does not mention that
edges entering the block are also in the same bundle (which seems true from
what the underlying code is doing).

Diff Detail

Event Timeline

mkazantsev requested review of this revision.Mar 22 2021, 10:23 PM
mkazantsev created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptMar 22 2021, 10:23 PM

Just a sanity check that I understood it correctly.

reames resigned from this revision.Mar 23 2021, 9:47 AM

Never looked at this code, not a qualified reviewer.

RKSimon accepted this revision.Mar 23 2021, 11:37 AM
RKSimon added a subscriber: stoklund.

Makes sense to me, but I don't think anyone other than @stoklund ever touched this code other than for general maintenance work.

This revision is now accepted and ready to land.Mar 23 2021, 11:37 AM
This revision was automatically updated to reflect the committed changes.