Skip to content

Enable dual precision flang builds - #157

Merged
awnawab merged 10 commits into
ecmwf-ifs:mainfrom
awnawab:naan-flang-dual-precision
Jul 30, 2026
Merged

Enable dual precision flang builds#157
awnawab merged 10 commits into
ecmwf-ifs:mainfrom
awnawab:naan-flang-dual-precision

Conversation

@awnawab

@awnawab awnawab commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator

This PR makes the field_api object libraries truly precision independent, thereby enabling dual precision builds for flang, as described in #138.

awnawab and others added 10 commits July 8, 2026 14:50
…s#114)

PR ecmwf-ifs#114 ("Remove unused symbols and imports from FIELD_DEFAULTS_MODULE")
introduced per-precision field_api_defaults_${prec} object libraries by
splitting field_defaults_module.F90 out of field_api_core
(commit be2bda3, "Compile field_defaults_module for each precision").

This reverts that change, folding field_defaults_module.F90 back into
field_api_core and restoring its GET_VIEW_ABORT/DELAYED compile
definitions and the PREC parameter on field_api_add_object_library.

It also reverts the dependent LLVMFlang workaround from PR ecmwf-ifs#137
(commit f07db36) that linked field_api_defaults_${DEFAULT_PRECISION}
into the buffer/factory/shuffle object libraries, since those targets
no longer exist.

The unrelated copyright-header commit (333cf7c) and the later
FIELD_API_COMPILE_DEFINITIONS / hipfort additions are preserved.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@awnawab
awnawab requested review from dareg and mlange05 July 27, 2026 11:01
@awnawab awnawab added the approved-for-ci Approved to run hpc-ci label Jul 27, 2026

@mlange05 mlange05 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.

Looks good to me.

@awnawab
awnawab merged commit 447c1a7 into ecmwf-ifs:main Jul 30, 2026
128 of 132 checks passed
@awnawab
awnawab deleted the naan-flang-dual-precision branch July 30, 2026 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved-for-ci Approved to run hpc-ci contributor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants