mikeoliphant / neural-amp-modeler-lv2

Neural Amp Modeler LV2 plugin implementation
GNU General Public License v3.0
227 stars 28 forks source link

Plugin causes crashes (x86_64 Linux) #50

Closed 38github closed 1 year ago

38github commented 1 year ago

REAPER (x86_64, OpenSUSE Slowroll) crashes using both git20231010 and git20231019 builds. I have tried both GCC 13 and Clang 17 but it happens with all of them. I got no useful information in the terminal that can help us. I have also tried both REAPER 6.83 and 7.001.

EDIT: Also crashes Ardour 7.5

mikeoliphant commented 1 year ago

Not sure what you mean by "git20231010". If you can isolate which commit caused the problem to start, that would be helpful.

Does the latest release version work for you?

Does it crash when you add the plugin with no model loaded, or just when you load a model (or load a preset with a model loaded)?

38github commented 1 year ago

Sorry. It appears that a compile flag caused this. I am sorry for taking up your time.

38github commented 1 year ago

Trying out openMP (maybe doesn't do anything spped wise) with GCC was ok but with Clang it caused all this. When just compiling it with "-O3 -march=x86-64-v3 flto=thin" it works as it should AND it uses less CPU than GCC 13.

Standard WaveNet: GCC 13: 3.4% Clang 17: 2.9%

mikeoliphant commented 1 year ago

Yeah, clang seems to do a better job of optimizing the underlying matrix operations.

The crash might be the compiler adding vectorization that your CPU doesn't support.

38github commented 1 year ago

Thank you for taking your time and I will do my best to not rush an ISSUE before I have thoroughly tested my setup. I am sorry for that.

mikeoliphant commented 1 year ago

No worries.