This is an archive of the discontinued LLVM Phabricator instance.

[DirectX] Fix two "not all control paths return a value" warnings
ClosedPublic

Authored by bogner on Jul 10 2023, 1:19 PM.

Details

Summary

These switch statements are fully covered. Add an llvm_unreachable so
that compilers that don't recognize that don't warn.

Diff Detail

Event Timeline

bogner created this revision.Jul 10 2023, 1:19 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 10 2023, 1:19 PM
bogner requested review of this revision.Jul 10 2023, 1:19 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 10 2023, 1:19 PM
beanz accepted this revision.Jul 10 2023, 1:22 PM

LGTM

This revision is now accepted and ready to land.Jul 10 2023, 1:22 PM
python3kgae accepted this revision.Jul 10 2023, 1:25 PM
This revision was landed with ongoing or failed builds.Jul 10 2023, 3:35 PM
This revision was automatically updated to reflect the committed changes.