Skip to content

Testset fixes based on Workitem 17032 (for GCC) - #142

Merged
ohno-fj merged 2 commits into
fujitsu:mainfrom
ohno-fj:fix/Workitem-17032
Sep 10, 2026
Merged

Testset fixes based on Workitem 17032 (for GCC)#142
ohno-fj merged 2 commits into
fujitsu:mainfrom
ohno-fj:fix/Workitem-17032

Conversation

@ohno-fj

@ohno-fj ohno-fj commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator

I will correct the testset based on "Workitem 17032".

Specifically, I will make the following correction.
The test assumes that STDC_IEC_559_COMPLEX is always defined when complex arithmetic support is available.
However, the macro is intentionally undefined under -ffast-math because IEEE 754/ISO IEC 60559 compliance is no longer guaranteed.

Based on the above, when assuming IEEE 754 compatible complex number arithmetic, I specify the following options.

  • -fno-finite-math-only
  • -fno-cx-limited-range
  • -fno-unsafe-math-optimizations

@ohno-fj
ohno-fj requested a review from itou-fj September 9, 2026 04:47
@ohno-fj ohno-fj added the bug Something isn't working label Sep 9, 2026
Comment thread C/0005/CMakeLists.txt Outdated
@ohno-fj

ohno-fj commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator Author

@itou-fj

I have fixed the issues pointed out in the following pull requests.
#142
#141

Specifically, I made the following changes:
I move the following lines

  • set_property(SOURCE 0005_0033.c APPEND PROPERTY COMPILE_OPTIONS "-std=c17") -> Workitem 17030
  • set_property(SOURCE 0005_0058.c APPEND PROPERTY COMPILE_OPTIONS "-fno-finite-math-only;-fno-cx-limited-range;-fno-unsafe-math-optimizations") -> Workitem 17032

between the line

  if(TEST_SUITE_FUJITSU_ENABLE_${llvm_test_lang}_NON_OPENMP)

and the line

  endif()

below it.

@ohno-fj
ohno-fj merged commit 6e79912 into fujitsu:main Sep 10, 2026
@ohno-fj
ohno-fj deleted the fix/Workitem-17032 branch September 10, 2026 05:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants