Page MenuHomePhabricator

[clang] Implement CompilerInvocation copy assignment
ClosedPublic

Authored by jansvoboda11 on Apr 14 2021, 5:40 AM.

Details

Summary

This patch implements the copy assignment for CompilerInvocation.

Eventually, the deep-copy operation will be moved into a clone() method (D100460), but until then, this is necessary for basic ergonomics.

Depends on D100455.

Diff Detail

Event Timeline

jansvoboda11 requested review of this revision.Apr 14 2021, 5:40 AM
jansvoboda11 created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptApr 14 2021, 5:40 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
Bigcheese accepted this revision.Apr 16 2021, 8:56 AM

I think this is fine given that we already have a copy constructor with deep copy semantics.

lgtm

This revision is now accepted and ready to land.Apr 16 2021, 8:56 AM
This revision was automatically updated to reflect the committed changes.