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 test subject print #30

Closed

Conversation

rouson
Copy link
Member

@rouson rouson commented Oct 27, 2024

This commit replaces list-directed printing of the test_t subject with '(*(a)) formatted printing to work around LLVM flang's relatively short wrapping length for list-directed printing.

ktras and others added 24 commits September 3, 2024 15:11
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.
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.
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.
@rouson rouson closed this Oct 27, 2024
@rouson rouson deleted the fix-test-subject-print branch October 27, 2024 16:01
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.

2 participants