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

Ported titanxstall test to moderngpu mergesort #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

seanbaxter
Copy link

I forked your repo and submoduled moderngpu into it. It's not quite literally what you had--my mergesort reuses the input array to ping-pong the passes, so it only accepts pointers, not iterators. So I have a transform() call that combines the hash and the particleinfo into a tuple<>. I believe thrust does this internally, as I cannot see how it can ping-pong between your provided zip-iterator and a by-value store that it creates on its side.

The hash is a function of counter to keep things changing. Anyways, I've run out to 25,000 iterations now.

@seanbaxter
Copy link
Author

One other thing is that the thrust build generates 220 STL instructions, when you look at the PTX. I think this is due to the way thrust implements tuples. Very heavy traffic into local memory doesn't mean the thing is broken but it does make me nervous. Past 44,000 iterations now.

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.

None yet

1 participant