Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix incorrect IR for expression with two ternary operators #220

Merged
merged 3 commits into from
Jul 15, 2024

Conversation

IlyaMuravjov
Copy link

@IlyaMuravjov IlyaMuravjov commented Feb 16, 2024

Fixes #219

Doesn't pass UnusedVariableTest, which seems to be less bad when compared to producing incorrect IR

@IlyaMuravjov IlyaMuravjov changed the title Add failing test for expression with two ternary operators Fix incorrect IR for expression with two ternary operators Feb 16, 2024
Copy link

codecov bot commented Feb 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 59.80%. Comparing base (2dc8efe) to head (dc2a778).

Additional details and impacted files
@@             Coverage Diff              @@
##                neo     #220      +/-   ##
============================================
- Coverage     59.91%   59.80%   -0.12%     
+ Complexity     2417     2416       -1     
============================================
  Files           294      294              
  Lines         18469    18469              
  Branches       3306     3306              
============================================
- Hits          11066    11045      -21     
- Misses         6328     6346      +18     
- Partials       1075     1078       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Saloed Saloed changed the base branch from develop to neo July 12, 2024 15:02
@IlyaMuravjov IlyaMuravjov force-pushed the ilya_m/two-ternary-operators branch from 4214090 to 59a7466 Compare July 12, 2024 15:41
@Saloed Saloed self-requested a review July 12, 2024 16:16
@Saloed Saloed merged commit 1aedf93 into UnitTestBot:neo Jul 15, 2024
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wrong IR for two ternary operators
2 participants