This is an archive of the discontinued LLVM Phabricator instance.

Demo fix for readability-redundant-control-flow (WIP)
AbandonedPublic

Authored by mehdi_amini on Dec 28 2021, 10:02 PM.

Details

Reviewers
rriddle
Mogball

Diff Detail

Event Timeline

mehdi_amini created this revision.Dec 28 2021, 10:02 PM
mehdi_amini requested review of this revision.Dec 28 2021, 10:02 PM
Mogball accepted this revision.Dec 29 2021, 8:46 AM
This revision is now accepted and ready to land.Dec 29 2021, 8:46 AM

By the word, I run this check over entire LLVM code base (but only X86 architecture) and there were only 8 detected occurrences. It make sense to fix them across entire code base and enable check on top level .clang-tidy.

rriddle accepted this revision.Jan 1 2022, 3:26 AM
mehdi_amini abandoned this revision.Jan 1 2022, 6:03 PM