-
Notifications
You must be signed in to change notification settings - Fork 23
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
Please update Eigen headers or at least disable Altivec for macOS until Eigen updated #1411
Comments
What version of nimble are you installing? A new version (1.1.0) is just going up on CRAN now. (It can take a couple of days for all binaries to be updating on CRAN, but the source is there.) That version does update Eigen to its current version of 3.4.0. If you are picking up nimble 1.1.0, then maybe that Eigen update is the problem? If you are using an earlier version of nimble, then maybe the new version is what you need? |
I.e., this gonna fix it (though in a suboptimal way):
|
@perrydv Thank you for responding. I am building 1.1.0. Eigen 3.4.0 has a broken Altivec. The fix was merged after that. |
@perrydv Altivec headers were fixed here: https://gitlab.com/libeigen/eigen/-/merge_requests/1150 I have built |
@perrydv Hmm, configure arg does not have any effect. How should it be used?
But it ignored the passed directory path:
(Obviously, I can patch in the flag, but can it be made to work in a neater way?) |
Well, I was able to make it recognize the argument, but build still violates it and uses its own Eigen:
But:
What should I do? :) |
Thanks @barracuda156 . This is very helpful. Eigen 3.4.0 is labeled as "latest stable release". How common will this problem be? @paciorek can say more about the configure args. |
@perrydv If there are any reservations with regard to updating Eigen headers, do not do it, it is not worth: affected people are obviously very few. (I suspect that nobody has working However it will be nice to either ensure that a build against external Eigen actually works without hacking the code (and a note in README, maybe, advising a usage of newer Eigen from upstream or Macports |
P. S. I will update the port for |
Yes I will check on the question of building against external Eigen via the configure args when I get a chance. |
@perrydv @paciorek My updates are merged, so now it hopefully can be reproducible: Macports will build While this version of the portfile is supposed to build against Macports Eigen: This issue as such is orthogonal to AltiVec breakage. If we make it work for x86 or arm64, it should work for any arch. By default Macports installs Eigen headers into |
nimble
uses an outdated version of Eigen headers, it seems, which misdetects VSX support in Altivec code, which breaks the build on macOS on PowerPC:Upstream Eigen had this fixed quite a while ago. If updating it problematic or undesirable, a quick fix would be to just disable Altivec for Apple.
The text was updated successfully, but these errors were encountered: