Intel Performance Primitives

[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.

[Adam_G]

@d-healey thank you

[d.healey]

Yeah that worked, here's the PR