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

Roberto Sassi's webpage udated #162

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

Conversation

inuritdino
Copy link
Contributor

An updated link to Sassi's webpage in the help of ecgpuwave. The new link is believed to be corresponding to the old one.

An updated link to Sassi's webpage in the help of ecgpuwave. The new link is believed to be corresponding to the old one.
@bemoody
Copy link
Collaborator

bemoody commented Jan 13, 2022

Thanks for pointing this out. This comment is somewhat out of date:

  • Nowadays, ecgpuwave is built with gfortran by default rather than with g77.

  • The Toolbox package includes ecgpuwave binaries for all supported platforms (which nowadays are all 64-bit.)

  • It's true that ecgpuwave on x86-64 produces slightly different results than ecgpuwave on x86-32. But, as far as I know, this is due to the use of SSE instructions rather than x87 (64-bit rather than 80-bit floats), not because of differences between gfortran and g77.

So I think we should simply remove this comment.

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