Skip to content

Update flash.cu#9

Open
eclouder wants to merge 1 commit intotspeterkim:mainfrom
eclouder:main
Open

Update flash.cu#9
eclouder wants to merge 1 commit intotspeterkim:mainfrom
eclouder:main

Conversation

@eclouder
Copy link

determine Bc, Br dynamically

determine Bc, Br dynamically
Copy link

@xavhl xavhl left a comment

Choose a reason for hiding this comment

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

Thanks for the helpful update! I tested your update and concluded with the following suggestions:

  1. divide by sizeof(float), or any other types, since prop.sharedMemPerBlock is in bytes
  2. min(shared/4d, N) to cater small input size
// line 91:
const int Bc = min(ceil(prop.sharedMemPerBlock/sizeof(float)/(4*d)), (float)N);

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