This is an archive of the discontinued LLVM Phabricator instance.

[Docs] Fix missing docs strings for CallingConv.h
ClosedPublic

Authored by marcauberer on Aug 4 2022, 2:45 AM.

Details

Summary

Replaces

//

with

///

for some code lines to make it visible in the auto-generated documentation.

Diff Detail

Event Timeline

marcauberer created this revision.Aug 4 2022, 2:45 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 4 2022, 2:45 AM
Herald added a subscriber: StephenFan. · View Herald Transcript
marcauberer requested review of this revision.Aug 4 2022, 2:45 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 4 2022, 2:45 AM
marcauberer edited the summary of this revision. (Show Details)Aug 4 2022, 2:46 AM
dblaikie accepted this revision.Aug 4 2022, 11:40 AM

Sounds great - thanks!

This revision is now accepted and ready to land.Aug 4 2022, 11:40 AM

Can you submit the change for me? I have no permissions for that.

MaskRay accepted this revision.Aug 4 2022, 12:00 PM

You can do "Don’t duplicate function or class name at the beginning of the comment." from https://llvm.org/docs/CodingStandards.html#commenting while updating the comments.

This revision was landed with ongoing or failed builds.Aug 4 2022, 12:15 PM
This revision was automatically updated to reflect the committed changes.

@MaskRay okay, too late. I'm gonna open a new change for this ...