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

Supplementary header file #157

Merged
merged 2 commits into from
Dec 29, 2023
Merged

Supplementary header file #157

merged 2 commits into from
Dec 29, 2023

Conversation

haoliang-Zhu
Copy link
Contributor

Supplement the missing header files in the workload-xsave folder.Modify the yogini source file and output help when input is missing.

@ysun
Copy link
Contributor

ysun commented Dec 28, 2023

Please re-write the description for both commits.
It's required to provide commit titles and descriptions in English.

@@ -284,6 +283,8 @@ static void cmdline(int argc, char **argv)
case 'f':
clfulsh = 1;
break;
case '?':
Copy link
Contributor

Choose a reason for hiding this comment

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

Hi Haoliang, incorrect indent here.

@@ -74,7 +74,7 @@ struct workload *(*all_register_routines[]) () = {
register_SSE,
register_MEM,
register_memcpy,
register_AMX,
s register_AMX,
Copy link
Contributor

Choose a reason for hiding this comment

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

Please remove 's'.

1.Modify u_int8_t to uint8_t.

Signed-off-by: Haoliang Zhu <[email protected]>
@haoliang-Zhu haoliang-Zhu force-pushed the cmake branch 8 times, most recently from c459d6e to 40db7a9 Compare December 29, 2023 07:26
1.Missing uint8_t header file.
2.Supplement useage information.

Signed-off-by: Haoliang Zhu <[email protected]>
@ysun ysun merged commit bc075f9 into intel:main Dec 29, 2023
1 of 2 checks passed
@haoliang-Zhu haoliang-Zhu deleted the cmake branch January 2, 2024 03:55
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