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

Improve make report with extra test arguments #1369

Merged
merged 1 commit into from
Nov 21, 2023
Merged

Conversation

kito-cheng
Copy link
Collaborator

Recently we have add the capability to adding extra compilation options for running GCC testsuite, but it seems not handle well on the make report part, it will ignore extra compilation options, and then...combine multiple report together.

Now it will put in two different line, and the extra compilation options will listed in follwoing line, here is example output here:

               ========= Summary of gcc testsuite =========
                            | # of unexpected case / # of unique unexpected case
                            |          gcc |          g++ |     gfortran |
 rv64gc_zicfilp_zicfiss/  lp64d/ medlow |20350 /  3967 |10158 /  2583 |18374 /  3098 |
 rv64gc_zicfilp_zicfiss/  lp64d/ medlow |   86 /    19 |   15 /     9 |   51 /    51 |
 -static

Recently we have add the capability to adding extra compilation options
for running GCC testsuite, but it seems not handle well on the `make report`
part, it will ignore extra compilation options, and then...combine
multiple report together.

Now it will put in two different line, and the extra compilation options
will listed in follwoing line, here is example output here:

               ========= Summary of gcc testsuite =========
                            | # of unexpected case / # of unique unexpected case
                            |          gcc |          g++ |     gfortran |
 rv64gc_zicfilp_zicfiss/  lp64d/ medlow |20350 /  3967 |10158 /  2583 |18374 /  3098 |
 rv64gc_zicfilp_zicfiss/  lp64d/ medlow |   86 /    19 |   15 /     9 |   51 /    51 |
 -static
@kito-cheng
Copy link
Collaborator Author

cc @Incarnation-p-lee, it's follow up patch of #1368

Copy link
Collaborator

@patrick-rivos patrick-rivos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Incarnation-p-lee
Copy link
Contributor

OK, thanks Kito for the follow-up.

@kito-cheng kito-cheng merged commit 6b60183 into master Nov 21, 2023
20 checks passed
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.

3 participants