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

Doesn't build on i386 #5

Open
trebmuh opened this issue Apr 1, 2020 · 8 comments
Open

Doesn't build on i386 #5

trebmuh opened this issue Apr 1, 2020 · 8 comments

Comments

@trebmuh
Copy link
Contributor

trebmuh commented Apr 1, 2020

Hi @brummer10 , trying to build on a Debian Buster on a i386 chroot, it gives me the following:

g++ -g -O2 -fdebug-prefix-map=/build/xmonk.lv2-0.4=. -fstack-protector-strong -Wformat -Werror=format-security -D_FORTIFY_SOURCE=2 -I. -I./dsp -I./plugin -fPIC -DPIC -O2 -Wall -funroll-loops -fstack-protector -ffast-math -fomit-frame-pointer -fstrength-reduce -fdata-sections -Wl,--gc-sections -Wl,-z,relro,-z,now -Wl,--exclude-libs,ALL -msse3 -mfpmath=sse plugin/Xmonk.cpp  -Wl,-z,relro -I. -shared -lm -Wl,-z,noexecstack -fvisibility=hidden -o Xmonk.so
plugin/Xmonk.cpp: In constructor 'xmonk::DenormalProtection::DenormalProtection()':
plugin/Xmonk.cpp:121:5: error: '__builtin_ia32_fxsave' was not declared in this scope
     __builtin_ia32_fxsave(&fxsave);
     ^~~~~~~~~~~~~~~~~~~~~
plugin/Xmonk.cpp:121:5: note: suggested alternative: '__builtin_ia32_xsave'
     __builtin_ia32_fxsave(&fxsave);
     ^~~~~~~~~~~~~~~~~~~~~
     __builtin_ia32_xsave
make[2]: *** [Makefile:99: Xmonk] Error 1
make[2]: Leaving directory '/build/xmonk.lv2-0.4/Xmonk'
make[1]: *** [Makefile:18: Xmonk] Error 2
make[1]: Leaving directory '/build/xmonk.lv2-0.4'

Any idea maybe?

@brummer10
Copy link
Owner

Could you try please with -mfxsr added to the CXXFLAGS in the makefile.
I don't have a i386 box, so I only guessing.

@trebmuh
Copy link
Contributor Author

trebmuh commented Apr 2, 2020

That did the trick.
I mean, with adding -mfxsr to the CXXFLAGS in Xmonk/Makefile, it built on a Debian Buster i386 chroot here, note that I didn't test the resulting binary on a i386 system, just it built (tm).
Thank you very much @brummer10 , closing here now.

@trebmuh trebmuh closed this as completed Apr 2, 2020
@brummer10
Copy link
Owner

Thank you for trying. So I could add a fix for this in general to all affected projects.
btw. you are aware of this one: https://github.com/brummer10/GxMatchEQ.lv2
as it use the same denormalize technic then this one here.

@trebmuh
Copy link
Contributor Author

trebmuh commented Apr 2, 2020

I'm not! Usually, I'm only using/packaging Guitarix and the GXplugins suite (plus screcord, gxtuner, and jalv.select). Is this GXmatchEQ supposed to be incorporated into the GXplugins at a later point? The reason I'm asking is because, I would probably avoid to package it for LZK if it is expected to join the GXplugins family (it's a bit of a pain to manage this king of transition).
BTW: do you have other plugins/project LA-related maybe?

@brummer10
Copy link
Owner

No, it will not join the GxPlugins suite. all GxPlugins been suited for the MOD, that is not the case for GxMatchEQ.
I've some other projects LA related, but nothing worse to package. So when you take that one, you'll have all worse to keep a eye on it. Note, for now it will have the same issue as this one here, I need to push a fix latter on.

@trebmuh
Copy link
Contributor Author

trebmuh commented Apr 2, 2020

OK, thanks for the time you took for that. I'll look at packaging this GxMatchEQ for LZK soon-ish.

@trebmuh
Copy link
Contributor Author

trebmuh commented Dec 30, 2021

Hi @brummer10 , just came back reading at this one.
It does seem that the "mfxsr" thingy has not made it into the master code here. Is that on purpose?

@trebmuh trebmuh reopened this Dec 30, 2021
@brummer10
Copy link
Owner

Yea, I've forgotten about it. Should be fixed now with the latest commit.

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

No branches or pull requests

2 participants