[clang] co_return cleanup
AcceptedPublic
Actions

Authored by urnathan on Fri, May 13, 6:07 AM.

Details

Reviewers

bruno
iains
MaskRay

Summary

There's no need for the CoreturnStmt getChildren member to
deal with the presence or absence of the operand member. All
users already deal with null children.

Diff Detail

Event Timeline

urnathan created this revision.Fri, May 13, 6:07 AM

Herald added a project: Restricted Project. · View Herald TranscriptFri, May 13, 6:07 AM

Herald added a subscriber: StephenFan. · View Herald Transcript

urnathan requested review of this revision.Fri, May 13, 6:07 AM

urnathan added a parent revision: D125541: [clang][PR55406] co_return CFG fix.

Harbormaster completed remote builds in B164297: Diff 429216.Fri, May 13, 7:01 AM

LGTM. (I need to learn this stuff but I'm happy since bots are happy:))

This revision is now accepted and ready to land.Fri, May 13, 11:04 PM

urnathan edited the summary of this revision. (Show Details)Tue, May 17, 12:00 PM

urnathan removed a parent revision: D125541: [clang][PR55406] co_return CFG fix.

Revision Contents
Changeset List

Path

Size

clang/

include/

clang/

AST/

StmtCXX.h

6 lines

Diff 429216

View Options

clang/include/clang/AST/StmtCXX.h

[clang] co_return cleanupAcceptedPublicActions