This is an archive of the discontinued LLVM Phabricator instance.

[CodeGen] Restrict addEmittedDeferredDecl to incremental extensions
ClosedPublic

Authored by Hahnfeld on Aug 8 2023, 2:29 AM.

Details

Summary

Reemission is only needed in incremental mode. With this early return,
we avoid overhead from addEmittedDeferredDecl in non-incremental mode.

Diff Detail

Event Timeline

Hahnfeld created this revision.Aug 8 2023, 2:29 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 8 2023, 2:29 AM
Hahnfeld requested review of this revision.Aug 8 2023, 2:29 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 8 2023, 2:29 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript

This LGTM but I'd like to wait for @rjmccall for a week or so.

v.g.vassilev accepted this revision.Aug 15 2023, 3:58 AM

@Hahnfeld, let's move forward and rely on a post-commit review here if necessary.

This revision is now accepted and ready to land.Aug 15 2023, 3:58 AM
rjmccall accepted this revision.Aug 15 2023, 12:57 PM

Thank you, this seems reasonable.