This is an archive of the discontinued LLVM Phabricator instance.

Explain the symbols of basic block clusters with an example in the header comments.
ClosedPublic

Authored by rahmanl on Jul 29 2021, 7:22 PM.

Details

Summary

This prevents from confusion with the `labels` option.

Diff Detail

Event Timeline

rahmanl created this revision.Jul 29 2021, 7:22 PM
rahmanl requested review of this revision.Jul 29 2021, 7:22 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 29 2021, 7:22 PM
snehasish accepted this revision.Jul 29 2021, 7:26 PM
snehasish added inline comments.
llvm/lib/CodeGen/BasicBlockSections.cpp
37

How about a minor edit here:

... basic block 4 (note that it is not referenced in the list) is placed in one section ...

This revision is now accepted and ready to land.Jul 29 2021, 7:26 PM
rahmanl updated this revision to Diff 362957.Jul 29 2021, 7:29 PM
rahmanl marked an inline comment as done.

Apply Snehasish's edit.