This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Replace an 'else if' with 'else'+assert. NFC
ClosedPublic

Authored by craig.topper on Mar 27 2023, 1:10 PM.

Details

Summary

There are only two cases here. Using an assert ensures there is
no handled third case.

Also move comment to avoid odd formatting.

Diff Detail

Event Timeline

craig.topper created this revision.Mar 27 2023, 1:10 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 27 2023, 1:10 PM
craig.topper requested review of this revision.Mar 27 2023, 1:10 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 27 2023, 1:10 PM
craig.topper retitled this revision from [RISCV] Replace an 'else if' with 'else'+assert. to [RISCV] Replace an 'else if' with 'else'+assert. NFC.Mar 27 2023, 1:10 PM
reames accepted this revision.Mar 27 2023, 1:30 PM

LGTM

This revision is now accepted and ready to land.Mar 27 2023, 1:30 PM
luke accepted this revision.Mar 27 2023, 1:42 PM
This revision was landed with ongoing or failed builds.Mar 27 2023, 1:48 PM
This revision was automatically updated to reflect the committed changes.