This is an archive of the discontinued LLVM Phabricator instance.

[memprof][NFC] Update a stale comment about symbolization.
ClosedPublic

Authored by snehasish on Apr 27 2023, 11:57 AM.

Details

Summary

Symbolization of position independent code was added in D146181. Update
the comment to note that the checks below are to sanity check the
assumptions we make to simplify symbolization.

Diff Detail

Event Timeline

snehasish created this revision.Apr 27 2023, 11:57 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 27 2023, 11:57 AM
Herald added a subscriber: hiraditya. · View Herald Transcript
snehasish requested review of this revision.Apr 27 2023, 11:57 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 27 2023, 11:57 AM
This revision was not accepted when it landed; it landed in state Needs Review.Apr 27 2023, 11:59 AM
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.