MetOffice / MetOffice/lfric_apps

Further Refactor of excf_nl_9c

Open
#571 0 comments 0 reactions 1 assignee Claimed by @MetBenjaminWent View on GitHub
enhancement
Dominant language
Fortran
Stars
31
Forks
118
Avg merge
2d 4h
Merged PRs (30d)
8

Description

### Are there any linked Issues or Pull Requests?

PR: #576
Umbrella: #106

### Brief description

This file has pushed the desire to avoid OMP syncronisation to the max, using multiple OMP master and barrier to effectively syncronise sections and fully span the parallel region over the whole file. While in an ideal world we want this, for porting source to other hardware and paralism methods, it might be worth diving the parallel sections up, and adjusting the source to be more simple.

The goal is to improve portability, but we may see a minor degrade in OMP performance. If it is very minor, than it will be acceptable and put to review. Otherwise we'll look at it again.

The removal of j in this file has already occurred, and so parallelism for LFRic on Mlian has improved. Genoa may perform differently.

### Further details of the issue.

_No response_

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.