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

0.90.2 fails to build: no matching member function for call to 'lerp_table_lookup_float_mask' #227

Open
yurivict opened this issue Apr 17, 2019 · 8 comments

Comments

@yurivict
Copy link

The FreeBSD port fails to build:

In file included from plugin.cpp:33:
./calf/organ.h:160:19: error: no matching member function for call to 'lerp_table_lookup_float_mask'
        return ph.lerp_table_lookup_float_mask(data, ORGAN_BIG_WAVE_SIZE - 1);
               ~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~
@yurivict
Copy link
Author

yurivict commented Jul 9, 2019

@kfoltman Could you please help with this? I can't update the FreeBSD port because of this regression.

@yurivict
Copy link
Author

yurivict commented Aug 2, 2019

0.90.3 still has this problem.
The problem exists when built with clang-8, gcc-8 builds it ok.

Please fix the clang build.

@kfoltman
Copy link
Contributor

kfoltman commented Aug 2, 2019

Which version of clang-8 is it? I can't reproduce this on Linux.

@yurivict
Copy link
Author

yurivict commented Aug 2, 2019

$ c++ -v
FreeBSD clang version 8.0.0 (tags/RELEASE_800/final 356365) (based on LLVM 8.0.0)

@kfoltman
Copy link
Contributor

kfoltman commented Aug 2, 2019

It compiles with version 8.0.1-+rc4-1 and 7.0.1-8 - perhaps 8.0.0 has some bugs/regressions that result in spurious errors.

Meanwhile, I've started trying to fix all the clang-8 warnings. Most are harmless, but I found a few potential bugs or at least things done not exactly right.

@kfoltman
Copy link
Contributor

kfoltman commented Aug 2, 2019

Actually, no, I had an outdated version, without recent changes by Markus.

I can reproduce it now - seems to be a template substitution problem, so might be easy enough to fix.

@kfoltman
Copy link
Contributor

kfoltman commented Aug 2, 2019

I just pushed a possible fix for the clang compilation error, plus some easy warning fixes.

@giuliomoro
Copy link

@kfoltman I just pushed a possible fix for the clang compilation error, plus some easy warning fixes.

Thanks, the latest git head as of today (0f45a43) does built with no errors with clang-3.9 on Debian Stretch, while the latest release (0.90.3) fails. A new bugfix release including whatever magic change you made above to fix the build error would be welcome!

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

3 participants