This is an archive of the discontinued LLVM Phabricator instance.

[NFC] Make some ObjectFormatType switches covering
ClosedPublic

Authored by hubert.reinterpretcast on Jun 25 2019, 6:57 AM.

Details

Summary

This patch removes the default case from some switches on llvm::Triple::ObjectFormatType, and cases for the missing enumerators are then added.

For UnknownObjectFormat, the action (llvm_unreachable) for the default case is kept.

For the other unhandled cases, report_fatal_error is used instead.

Diff Detail

Repository
rL LLVM

Event Timeline

Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptJun 25 2019, 6:57 AM
Herald added subscribers: jsji, aheejin. · View Herald Transcript
sfertile accepted this revision.Jul 4 2019, 7:56 AM

LGTM.

This revision is now accepted and ready to land.Jul 4 2019, 7:56 AM
This revision was automatically updated to reflect the committed changes.