[refactoring] idea: convert a ternary operator into an if statement and vice-versa

[hyp]

Bugzilla Link	35000
Version	trunk
OS	All
CC	@TH3CHARLie

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 ternary operator into an if statement and vice-versa. For example, this statement:

  int x = expression ? a : b;

Can become:

    int x;
    if (expression) {
        x = a;
    } else {
        x = b;
    }

and vice-versa.

[TH3CHARLie]

Hi Alex, is this already taken by anyone or solved? If not, I'd like to give a try!

[elhewaty]

Hello, I am a beginner who is interested in contributing to LLVM.
Can anyone guide me through this?

[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

[rachejazz]

So I see many opted for working on this before, but as I read on other issues, as long as I send a patch I can get a review?

[EugeneZelenko]

@rachejazz: Sure, patch is best way to proceed!

[elhewaty]

@rachejazz, Yes, and feel free to work on it, my comment was a long time ago.
The sign of work is the assignee tag and the pull requests.

[DheerajAgarwal1234]

My team would like to work on this one

[bhanuponguru]

Hi,

I would like to work on this issue, I request you to assign this issue to me.

Thank you.

[harsh241082]

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

[zwuis]

vbvictor and I agree that it would be better to make it a clang-tidy check.

A WIP PR: #166822.