Skip to content

(towards #2971) move omp taskloop trans#2977

Merged
LonelyCat124 merged 14 commits into
masterfrom
2971_move_remaining_trans
Jul 16, 2025
Merged

(towards #2971) move omp taskloop trans#2977
LonelyCat124 merged 14 commits into
masterfrom
2971_move_remaining_trans

Conversation

@victoria-atkinson

Copy link
Copy Markdown
Collaborator

No description provided.

@codecov

codecov Bot commented Jul 1, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.92%. Comparing base (80b372f) to head (2d76b32).
Report is 15 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2977   +/-   ##
=======================================
  Coverage   99.92%   99.92%           
=======================================
  Files         365      366    +1     
  Lines       51716    51721    +5     
=======================================
+ Hits        51675    51680    +5     
  Misses         41       41           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@victoria-atkinson victoria-atkinson changed the title (towards #2971) move remaining transformations (towards #2971) move omp taskloop trans Jul 1, 2025

@LonelyCat124 LonelyCat124 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @victoria-atkinson

Couple of very small changes - one which I'd like a response from Sergi else we can leave it alone.

One other request I have is could you keep an list in #2668 of transformations that have been moved out of transformations.py, as we can start converting them to the new option functionality once they've been moved.

Comment thread examples/gocean/eg1/openmp_taskloop_trans.py Outdated
Comment thread src/psyclone/psyir/transformations/omp_taskloop_trans.py Outdated
Comment thread src/psyclone/tests/psyir/transformations/omp_taskloop_trans_test.py Outdated

@LonelyCat124 LonelyCat124 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests all pass, looks good, thanks.

@LonelyCat124 LonelyCat124 merged commit af5cb94 into master Jul 16, 2025
11 checks passed
@LonelyCat124 LonelyCat124 deleted the 2971_move_remaining_trans branch July 16, 2025 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants