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

Project 2: Xuntong Liang #1

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

PacosLelouch
Copy link

  • Repo Link
  • I implemented algorithms include scan, stream compaction, and radix sort. I completed a performance analysis and optimizations.
    • Implemented scan with CPU's implementation, Naive, Work-Efficient, and a shared memory optimization version Work-Efficient-Hierarchical, and compared with thrust::exclusive_scan.
    • Implemented stream compaction with CPU's implementations, Work-Efficient, and a shared memory optimization version Work-Efficient-Hierarchical, and compared with thrust::remove_if.
    • Implemented sort with shared memory optimization Work-Efficient-Hierarchical, and compared with CPU's std::sort and thrust::sort.
    • Completed a performance analysis and guessed what might be happening inside the Thrust's implementations.
  • Feedback: It is challenging to optimize the GPU programs with shared memory with the thought of accessing the memory parallel for me, but it is interesting and meaningful. There are few documents and blogs talking about the bank conflict. I think I should take some time to make a clear understanding on it to write better GPU programs. I am also very confused about what happens inside the Thrust's function. I believe the authors use some techniques and optimized for a long time, and maybe there are some magic functions called by them?

zixin96 pushed a commit to zixin96/Project2-Stream-Compaction that referenced this pull request Sep 21, 2021
zixin96 pushed a commit to zixin96/Project2-Stream-Compaction that referenced this pull request Sep 22, 2021
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.

1 participant