This is an archive of the discontinued LLVM Phabricator instance.

[DwarfDebug][NFC] Factor out 'isInitialized' logic
ClosedPublic

Authored by fdeazeve on Aug 21 2023, 9:31 AM.

Details

Summary

The class 'DbgVariable' can be in one of three states, and the "is any of them
initialized" logic for them is repeated in a couple of places. We may want to
expand this class in the future; as such, we factor out this common logic so
that it is easier to modify.

Diff Detail

Event Timeline

fdeazeve created this revision.Aug 21 2023, 9:31 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 21 2023, 9:31 AM
Herald added a subscriber: hiraditya. · View Herald Transcript
fdeazeve requested review of this revision.Aug 21 2023, 9:31 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 21 2023, 9:31 AM
fdeazeve edited the summary of this revision. (Show Details)Aug 21 2023, 9:40 AM
aprantl accepted this revision.Aug 21 2023, 9:53 AM

I wonder if these should all be constructors or static ::create() methods?

This revision is now accepted and ready to land.Aug 21 2023, 9:53 AM

I wonder if these should all be constructors or static ::create() methods?

I very much want to address this if we decide to split this class!

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