New `ggml_are_same_shape(src0, src1)` assert after upgrading ggml library

[LostRuins]

Creating an issue to follow up on #1144 (comment)

I was trying to port some old code in TTS.cpp that was using an earlier version of the ggml library to a newer version of ggml, and I'm running into a new assert after this pull.

Specifically, there is now a constraint for ggml_are_same_shape(src0, src1) at

ggml/src/ggml-cpu/binary-ops.cpp

Line 65 in 2621d5a

GGML_ASSERT(ggml_are_same_shape(src0, src1));

that constraint was not there before, for example in ggml_compute_forward_mul_f32

ggml/src/ggml-cpu/ggml-cpu.c

Line 5432 in 4b344ba

static void ggml_compute_forward_mul_f32(

May I check what should I change in order to avoid this assert? Is there a reason for this new limitation?

Thanks!

[LostRuins]

Update: Removing that assert seems to allow inference to proceed normally.

As far as I can tell, the output still seems correct. Although I would still like to know why that assert was required.

[slaren]

You can use ggml_cont() on the second parameter of ggml_mul to make it contiguous and avoid this error. I don't think that ignoring the assert is going to work.