Skip to content

Conversation

@FreeBirdLjj
Copy link
Contributor

More effective by following changes:

  • Add const to some parameters
  • Add shared() section to OpenMP pragma
  • Move some calculations into inner loop so some temp array can be replaced by temp elements.

@myint myint merged commit 8081990 into myint:master Jun 12, 2015
@myint
Copy link
Owner

myint commented Jun 12, 2015

Cool, thanks.

I've merged everything except a few of the unnecessary const parameters in the headers. (I completely agree that const parameters in the function implementation is very useful. But in the declaration I think it is unnecessary and possibly noisy to the reader.)

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