Skip to content

Update to Eigen 3.2.8 and fix UBSAN error #29

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

Merged
merged 3 commits into from
Feb 23, 2016

Conversation

yixuan
Copy link
Collaborator

@yixuan yixuan commented Feb 23, 2016

This PR imports Eigen 3.2.8 and applies the upstream patch to solve UBSAN errors. All dependent packages except gpuR have passed the check.

@eddelbuettel
Copy link
Member

Lovely work, as usual -- really appreciate it.

May run another rev.dep check for good measure and ship it then.

eddelbuettel added a commit that referenced this pull request Feb 23, 2016
Update to Eigen 3.2.8 and fix UBSAN error
@eddelbuettel eddelbuettel merged commit 058e8ba into RcppCore:master Feb 23, 2016
@yixuan
Copy link
Collaborator Author

yixuan commented Feb 23, 2016

Sounds good! Hope this version solves the problem.

@eddelbuettel
Copy link
Member

Hope so too; it is in the queue at CRAN now. Take a look at this which was way overdue ...

@yixuan
Copy link
Collaborator Author

yixuan commented Feb 23, 2016

Wow, just noticed that. Thank you so much!

@yixuan
Copy link
Collaborator Author

yixuan commented Feb 24, 2016

Hmm, seems another one: http://www.stats.ox.ac.uk/pub/bdr/memtests/clang-UBSAN/RcppEigen/tests/doRUnit.Rout.
Let me see if we have any quick fix.

@eddelbuettel
Copy link
Member

Ugggh. Thanks for looking into it!

@yixuan
Copy link
Collaborator Author

yixuan commented Feb 24, 2016

This should not affect dependent packages much, though. It appears in a sparse Cholesky decomposition, which is not so frequently used in reverse dependencies.

@eddelbuettel
Copy link
Member

OTOH among the 50-ish packages using RcppEigen, quite a few do so because of sparse matrix algebra which RcppArmadillo is/was lacking...

@yixuan
Copy link
Collaborator Author

yixuan commented Feb 24, 2016

I've submitted a bug: http://eigen.tuxfamily.org/bz/show_bug.cgi?id=1172.
Sorry about that. I should have checked the package with UBSAN flags. :-\

@eddelbuettel
Copy link
Member

👍 and no worries. We squash them one by one.

When you hear back we can fold that into a 0.3.2.8.1 release.

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