-
-
Notifications
You must be signed in to change notification settings - Fork 364
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
[i386] miniaudio.h:83477:no such instruction:
lzcntl %eax, %ecx'`
#842
Comments
Thanks. This is an awkward one. So this is actually coming from dr_flac, which I amalgamate into miniaudio using a tool. It's used in a tight loop deep in the decoding logic and is important that it's used if available. dr_flac will do a runtime support check to ensure the instruction is not executed if unavailable, so I don't think it's causing any harm in practice at runtime, but I'm not sure how to disable this error. I want it to still be compiled into the executable even if the compiler is targeting i386. Do you have any suggestions? Is compilation actually succeeding, or is this presenting as an error instead of a warning? If it's a warning, since I'm doing a runtime check, I suppose silencing the warning work? |
@mackron It causes the build to fail, but admittedly this is a minority case, and it should not be disabled for everyone because it fails here. I do not really have a neat solution here. In principle, it should be checked that an instruction from a specific SIMD type can be actually compiled (instead of using a macro to check GCC/Clang version or check support for passing Given that |
@mackron In addition to #841 on older
i386
this may fail:Looks like there is no check there, it is assumed it is supported with all systems using gcc or clang, which is not the case: at least assembler has to support this insn, and it may not.
Build run on 10.6 i386 with gcc13, physical cpu CoreDuo.
The text was updated successfully, but these errors were encountered: