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.

Sign up for GitHub

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

Jump to bottom

fix: Align _mm_prefetch behavior to document #550

Merged
merged 1 commit into from
Nov 2, 2022
Merged

fix: Align _mm_prefetch behavior to document #550

merged 1 commit into from
Nov 2, 2022

Conversation

howjmay
Copy link
Collaborator

@howjmay howjmay commented Oct 30, 2022

Fixes #547

jserv
jserv reviewed Oct 30, 2022
View reviewed changes
jserv
jserv reviewed Oct 30, 2022
View reviewed changes
jserv
jserv reviewed Oct 30, 2022
View reviewed changes
jserv
jserv reviewed Oct 30, 2022
View reviewed changes
jserv
jserv reviewed Oct 30, 2022
View reviewed changes
jserv
jserv reviewed Oct 30, 2022
View reviewed changes
@howjmay howjmay force-pushed the fix-prefetch branch 2 times, most recently from dbba37c to 023f8a0 Compare October 30, 2022 13:42
jserv
jserv reviewed Nov 2, 2022
View reviewed changes
@howjmay howjmay force-pushed the fix-prefetch branch 2 times, most recently from 46e3170 to 6ae9e20 Compare November 2, 2022 10:31
@howjmay
Copy link
Collaborator Author

howjmay commented Nov 2, 2022

@jserv I removed some hints, since they are Knights Corner instead of SSE.
The followings are removed hints

    _MM_HINT_ENTA = 4, /* exclusive version of _MM_HINT_NTA */
    _MM_HINT_ET0 = 5,  /* exclusive version of _MM_HINT_T0 */
    _MM_HINT_ET1 = 6,  /* exclusive version of _MM_HINT_T1 */
    _MM_HINT_ET2 = 7   /* exclusive version of _MM_HINT_T2 */

@howjmay howjmay marked this pull request as ready for review November 2, 2022 11:31
@howjmay howjmay requested a review from marktwtn as a code owner November 2, 2022 11:31
@jserv jserv merged commit a387dd5 into DLTcollab:master Nov 2, 2022
@howjmay howjmay deleted the fix-prefetch branch November 5, 2022 07:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jserv jserv jserv left review comments

@marktwtn marktwtn Awaiting requested review from marktwtn

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Consolidate _mm_prefetch
2 participants
@howjmay @jserv