This is an archive of the discontinued LLVM Phabricator instance.

[SelectionDAG] Rename CallSiteDbgInfo to NodeExtraInfo
ClosedPublic

Authored by melver on Aug 1 2022, 1:56 AM.

Details

Summary

For information infrequently attached to SDNodes, it is useful to
provide a way to add this information out-of-line. This is already done
for call-site specific information.

Rename CallSiteDbgInfo to NodeExtraInfo in preparation of adding
additional information not necessarily related to call sites only.

Depends on D130876

Diff Detail

Event Timeline

melver created this revision.Aug 1 2022, 1:56 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 1 2022, 1:56 AM
melver requested review of this revision.Aug 1 2022, 1:56 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 1 2022, 1:57 AM

I looked at this and it looks good to me, but I don't feel like the right person to approve this.

melver updated this revision to Diff 450025.Aug 4 2022, 9:56 AM
melver edited the summary of this revision. (Show Details)

Rebase.

vitalybuka accepted this revision.Aug 5 2022, 5:02 PM
vitalybuka added inline comments.
llvm/include/llvm/CodeGen/SelectionDAG.h
275

SD is redundant as it's nested to SelectionDAG

This revision is now accepted and ready to land.Aug 5 2022, 5:02 PM
melver updated this revision to Diff 450854.Aug 8 2022, 10:02 AM
melver marked an inline comment as done.
melver retitled this revision from [SelectionDAG] Rename CallSiteDbgInfo into SDNodeExtraInfo to [SelectionDAG] Rename CallSiteDbgInfo to NodeExtraInfo.
melver edited the summary of this revision. (Show Details)

s/SDNodeExtraInfo/NodeExtraInfo/

This revision was automatically updated to reflect the committed changes.