This is an archive of the discontinued LLVM Phabricator instance.

[dsymutil] Don't print swift interface timestamp warning if we have no timestamp
ClosedPublic

Authored by JDevlieghere on Jan 14 2022, 9:48 AM.

Details

Summary

If we have no timestamp, don't print the corresponding warning. The binary holder already successfully ignores these cases, but the warning for swift interface files was lacking it.

rdar://86036385

Diff Detail

Event Timeline

JDevlieghere requested review of this revision.Jan 14 2022, 9:48 AM
JDevlieghere created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptJan 14 2022, 9:48 AM
aprantl accepted this revision.Jan 19 2022, 9:11 AM
aprantl added inline comments.
llvm/tools/dsymutil/DwarfLinkerForBinary.cpp
417–418

We could have a local variable called epochStart or something to make this more readable, but wither way is fine.

This revision is now accepted and ready to land.Jan 19 2022, 9:11 AM
This revision was landed with ongoing or failed builds.Jan 19 2022, 9:52 PM
This revision was automatically updated to reflect the committed changes.