Intel Performance Primitives

[Adam_G]

@d-healey ah nope let me give that a try

[Adam_G]

actually yes static library was enabled, i changed to Yes and the error is the same @d-healey

[Adam_G]

switching back to the deprecated library has worked, im pretty sure my old installation still existed but might be because i installed oneApi

[Adam_G]

now im getting C1083: Cannot open in
clude file: 'ippcore.h': No such file or directory during the vst compile process

[Adam_G]

pretty sure i need it, im drawing fft and getting some static. should i try an older hise version?

[d.healey]

@Adam_G said in Intel Performance Primitives:

should i try an older hise version?

Only if you're not using anything in your project that is in a more recent version.

I think it would be better to try to solve the real problem though.

From what you described above it's possible you have more than one version of IPP on your system, that's likely to cause problems. I'd clear off everything and start fresh with the OneAPI. Also try different options in Projucer, I'm not sure static linking is the option that's needed.

[Adam_G]

@d-healey okay thanks for the guidance ill try it a couple different things

[d.healey]

I've just tried building with the latest oneAPI version and I'm running into the same errors as you...

[d.healey]

Ok I solved it, looks like Intel moved their paths again or something.

Add this in Header Search Paths: C:\Program Files (x86)\Intel\oneAPI\ipp\latest\include\ipp
and this in Extra Library Search Paths: C:\Program Files (x86)\Intel\oneAPI\ipp\latest\lib

[Adam_G]

@d-healey said in Intel Performance Primitives:

C:\Program Files (x86)\Intel\oneAPI\ipp\latest\lib

thanks! that let me compile hise with oneapi default linking but im still getting error C1083: Cannot open include file: 'ipp.h': No such file or directory when i try compiling a vst with ipp :(

[d.healey]

@Adam_G Possible it needs to be added here too, in which case I don't think this is a good solution and I have an idea for another which I'll investigate now

[d.healey]

Ok I think I solved it, it's an issue with the IPP integration with VS

In Visual Studio:
Go to Tools > Options > Intel Compilers and Libraries
Next to Intel IPP, change the selected library to Intel Integrated Performance Primitives 2022.1 (or whatever version you have)

Let me know if it works...

Nope that didn't do it :(

[Adam_G]

@d-healey a brain bender! following along ive tried all the above. ive also tried including the paths in the Additional Include Directories in the project properties (Project -> [project name] Properties, under C/C++ | General). but ho change

[d.healey]

@Christoph-Hart @Adam_G I've found the root cause of the issue I think...

There are a few places like this in the source code

#if IPP_VERSION_MAJOR >= 2021 && IPP_VERSION_MINOR >= 10
#include <ipp/ippcore.h>
#include <ipp/ipps.h>
#else
 #include <ippcore.h>
 #include <ipps.h>
#endif
 #define JUCE_IPP_AVAILABLE 1
#endif

And the version check is not working. IPP_VERSION_MAJOR has been set correctly to 2022 but the minor version is only 1 so it's going to the else and looking for ippcore.h instead of ipp/ippcore.h

If I replace the paths then it "seems" to work.

[Adam_G]

@d-healey ahhhh wow that was very impressive detective work!

[d.healey]

@Adam_G said in Intel Performance Primitives:

wow that was very impressive detective work!

I just clicked the error message in visual studio ;)

[Adam_G]

@d-healey im watching a master at work

[d.healey]

@d-healey said in Intel Performance Primitives:

#if IPP_VERSION_MAJOR >= 2021 && IPP_VERSION_MINOR >= 10

Looks like this was Christoph's attempt to support newer versions of IPP. @Christoph-Hart Please fix :D

My suggested fix

#if (IPP_VERSION_MAJOR >= 2021 && IPP_VERSION_MINOR >= 10) || IPP_VERSION_MAJOR >= 2022

[Adam_G]

could you point me where to edit oh wise one @d-healey

[d.healey]

@Adam_G When building HISE just click each error message and it will jump you to the relevant spots in the source code. There were only 3 I think.

I'll try my suggested fix tomorrow, if Christoph doesn't get there first, and make a pull request.