Merge berkeleylab fork upstream #31
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add LLVM Flang to CI
chore(fpm.toml): update version
In order to test command_line_t, the test suite must run fpm via execute_command_line. With flang-new, this requires ensuring that the fpm flags are passed along to the so-invoked fpm commands. This commit updateds the README to direct uses to set FPM_FC and FPM_FFLAGS to ensure that the fpm commands launched by the test suite receive the necessary flags.
This commit edits the flang CI test script to follow the steps described in the README.md in commit 09f0fcd.
Fix n doc test workflow
doc(README): instruct to export FPM_{FC,FFLAGS)
This commit works around flang issue 109589, which causes line wraps after the 78th character of list-directed output. See llvm/llvm-project#109589.
Use language-feature macros and work around flang I/O issue
Read string_t-array value from JSON key/value pair
This commit adds
1. A unary operator(.csv.) that produces a string_t object
encapsulating a comma-separated value (CSV) created from
a. a string_t array or
b. a character array,
2. A binary operator(.sv.) that produces a string_t object
encapsulating a comma-separated value (CSV) created from
left- and right-hand sides, respectively corresponding to
a. a string_t array and a string_t separator or
b. a string_t array and a character separator.
c. a character array and a character separator
d. a character array and a string_t separator
3. A unit test that verifies the following identities:
"a,bc,def" == .csv. [string_t("a"), string_t("bc"), string_t("def")]
"abc,def" == .csv. ["abc", "def"]
"do|re|mi" == (string_t(["do", "re", "mi"]) .sv. "|" )
"dore|mi" == (([string_t("dore"), string_t("mi")]) .sv. string_t("|"))
"do|re|mi" == ( ["do", "re", "mi"] .sv. "|" )
"do|re|mi" == ( ["do", "re", "mi"] .sv. string_t("|"))
corresponding to cases 1a-b and 2a-d, respectively.
String bracketing csv json array
This commit replaces list-directed printing of the test_t subject with formatted printing to work around flang's relatively short wrapping length for list-directed printing.
fix(test_t): prevent subject wrapping in report
This commit switchs the write_lines type-bound procedure to using formatted output and thereby eliminates line wrapping caused by LLVM flang's line-length limit on list-directed output.
Fix: eliminate line wrap in file_t write_lines
This furthers a pattern of "test_" prefixes in the names of most of the user-facing features that directly support unit testing.
This commit adds unit tests of the search capability employed when a user runs a subset of the test suite by passing a --contains --contains argument to the test/main.F90 program.
This commit adds an example test suite that can be run with the command: fpm run --example main The resulting output demonstrates the recently capability for including diagnostic information in test output.
Feature: produce diagnostic output for test failures
This commit adds specific procedures supporting the string_t generic interface. The new procedures take dummy arguments of 1. logical: default-kind and c_bool 2. complex: default-kind and double-precision The commit also includes unit tests for each of the above four type/kind combinations.
Feature: construct `string_t` objects from logical & complex arguments
(docs): Fix link to example directory in README
This commit uses the NAG compiler's predefined NAGFOR macro to remove two lines in test_description_test.F90 to work around a bug in the nagfor compiler. The compiler bug has been isolated and reported and the associated macro conditionals can be removed once a patched version of the compiler is available. In the interim, simply substituting character variables for string_t arguments in the test's contains_text check works around the issue.
test(test_description_t): temp nagfor workaround
Update dependency Assert to 2.0.1
Fix: work around gfortran 14.2.0 bug
This commit fixes a problem with line continuation in string_test.F90 identified by the NAG compiler.
test(string_t): fix line continuation
The string_t constructors that accept real, double precision, complex, and double-precisions complex arguments previously used g0 edit descriptors that produced processor-dependent representations, breaking some of the corresponding tests when building with ifx. With this commit, all tests pass with ifx version 2025.0.4 Build 20241205.
Fix edit descriptors and add ifx build command to README.md
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.