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

Fixes for 64bit indices #4429

Merged
merged 3 commits into from
Dec 17, 2014
Merged

Fixes for 64bit indices #4429

merged 3 commits into from
Dec 17, 2014

Conversation

jwpeterson
Copy link
Member

This branch gets MOOSE compiling and running when PETSc is configured with --with-64-bit-indices=1, and libmesh is configured with --with-dof-id-bytes=8.

This commit shouldn't break anything for anyone using 32-bit indices, it's basically enforcing consistent use of dof_id_type over unsigned int where necessary. By the same token, new invalid uses of unsigned int will probably continue to creep into the code unless we're really vigilant, so it might be worth creating a "big indices" build target going forward...

Refs #4321.

John Peterson and others added 3 commits December 17, 2014 13:25
--with-dof-id-bytes configurations.

Restricting the aux_nonlinear_solution_from_xdr to only run when
libmesh has been configured with 4-byte dof ids.

Refs idaholab#4321.
@moosebuild
Copy link
Contributor

Results of testing 9796318 using moose_PR_pre_check recipe:

Passed on: linux-gnu

View the results here: https://www.moosebuild.com/view_job/9723

@moosebuild
Copy link
Contributor

Results of testing 9796318 using moose_PR_test recipe:

Passed on: linux-gnu

View the results here: https://www.moosebuild.com/view_job/9724

@moosebuild
Copy link
Contributor

Results of testing 9796318 using moose_PR_test_dbg recipe:

Passed on: linux-gnu

View the results here: https://www.moosebuild.com/view_job/9725

permcody added a commit that referenced this pull request Dec 17, 2014
@permcody permcody merged commit 20ea4e5 into idaholab:devel Dec 17, 2014
@jwpeterson jwpeterson deleted the 64bit branch December 17, 2014 23:43
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.

3 participants