[refactoring] idea: convert a switch statement to an if

[hyp]

Bugzilla Link	35004
Version	trunk
OS	All
CC	@m4tx

Extended Description

This is a possible idea for a new refactoring action for clang-refactor & editors that support integration with Clang's refactoring engine.

This idea suggests adding a refactoring that can convert a switch statement into an if statement. For example, this statement:

switch (x) {
case 0:
foo();
break;
case 1:
bar();
break;
}

Can become:

if (x == 1) {
    foo();
} else if (x == 2) {
    bar();
}

(Note: I'm intending to add an operation that implements the reverse, i.e. convert if to switch).

[hyp]

assigned to @m4tx

[llvmbot]

Hi!

This issue may be a good introductory issue for people new to working on LLVM. If you would like to work on this issue, your first steps are:

1. Assign the issue to you.
2. Fix the issue locally.
3. Run the test suite locally.
   3.1) Remember that the subdirectories under test/ create fine-grained testing targets, so you can
   e.g. use make check-clang-ast to only run Clang's AST tests.
4. Create a git commit
5. Run git clang-format HEAD~1 to format your changes.
6. Submit the patch to Phabricator.
   6.1) Detailed instructions can be found here

For more instructions on how to submit a patch to LLVM, see our documentation.

If you have any further questions about this issue, don't hesitate to ask via a comment on this Github issue.

@llvm/issue-subscribers-good-first-issue

[harsh241082]

Hi!
I would like to work on this issue. Please assign it to me.

[zwuis]

to become a standalone tool in clang, we need to go through RFC process, assign a maintainer (must be llvm member) and so on.. The best option for AST-based refactoring ideas is to be implemented in clang-tidy under readability/modernize section.

#34348 (comment)

Also removing good-first-issue because making a PR which will be 500+ lines (including check, tests, docs) is not a good way to start contributing. If anyone wishes to start working on linter tools (clang-tidy), I can find better good-first-issues.

#34348 (comment)

[llvmbot]

@llvm/issue-subscribers-clang-tidy

Author: Alex Lorenz (hyp)

| | | | --- | --- | | Bugzilla Link | [35004](https://llvm.org/bz35004) | | Version | trunk | | OS | All | | CC | @m4tx |

Extended Description

This is a possible idea for a new refactoring action for clang-refactor & editors that support integration with Clang's refactoring engine.

This idea suggests adding a refactoring that can convert a switch statement into an if statement. For example, this statement:

switch (x) {
case 0:
foo();
break;
case 1:
bar();
break;
}

Can become:

if (x == 1) {
    foo();
} else if (x == 2) {
    bar();
}

(Note: I'm intending to add an operation that implements the reverse, i.e. convert if to switch).