Makefile uses -march=native unconditionally, it will break building on PowerPC where it is not supported

[barracuda156]

On PowerPC -mtune=native and -mcpu=native are supported but not -march=native. This should be fixed in Makefile: https://github.com/LostRuins/koboldcpp/blob/concedo/Makefile

CMakeLists have correct flags but wrong arch detection, which will omit ppc: https://github.com/LostRuins/koboldcpp/blob/concedo/CMakeLists.txt
Correct check will be elseif (${CMAKE_SYSTEM_PROCESSOR} MATCHES "ppc|powerpc") (it does not matter if it is 32- or 64-bit, same flags are used).

[LostRuins]

Hi, can you please verify if ppc is actually using -march=native unconditionally? In the makefile, I am checking

ifeq ($(UNAME_M),$(filter $(UNAME_M),x86_64 i686 amd64))

that is the only branch that triggers -march=native
which should NOT trigger on a ppc device.

Can you verify if this is a new issue or did it exist before? And if it's still triggering, can you please share the env var values displayed in terminal during the make process here.

$(info I koboldcpp build info: )
$(info I UNAME_S:  $(UNAME_S))
$(info I UNAME_P:  $(UNAME_P))
$(info I UNAME_M:  $(UNAME_M))
$(info I UNAME_O:  $(UNAME_O))
$(info I CFLAGS:   $(CFLAGS))
$(info I CXXFLAGS: $(CXXFLAGS))
$(info I LDFLAGS:  $(LDFLAGS))
$(info I CC:       $(CCV))
$(info I CXX:      $(CXXV))
$(info )

As for the cmake file, it only supports building for CUDA on windows, you should not use it. Please make sure you're on the latest experimental branch.

[LostRuins]

Is this resolved?

[barracuda156]

@LostRuins I beg a pardon, perhaps I looked at the makefile without getting a coffee. You are right, of course, that code won’t apply on powerpc.

uname -m returns Power Macintosh on Darwin on PowerPC, uname -p returns powerpc. I think uname -o is invalid, but need to check when I am back to the machine.

[LostRuins]

Alright, do a test and let me know if the makefile has errors. If there are errors, do post the console output with the specific error and line number. Thanks.