Cambié algunas cosas, la función la puse arriba del `main`, hice que devuelva un **entero** (`int`) en vez un `void`, y que no reciba el segundo argumento. También hice que no sea necesario especificar la longitud del array, ahora lo detecta con `sizeof`. Además el main debe devolver un entero de valor `0` para verificar que se ejecutó correctamente.

Hacer esto, `int ventasSemanales[semanas]` está mal, no es la manera de recibir el argumento `semanas`, y además no es necesario, ya es posible conocer la longitud de tal array.

Código final:

    int semanaMenorVent(int ventasSemanales[],int longitud)
    {
        int semanaMenor = -1;
        int numeroMenor = ventasSemanales[0];
        int i;
        for (i = 0; i<longitud; i++)
        {
            if (ventasSemanales[i] < numeroMenor)
            {
                numeroMenor = ventasSemanales[i];
                semanaMenor = i;
            }
            printf("%i\n", semanaMenor);
        }
        return semanaMenor;
    }
    int main()
    {
        int ventasSemanales[]={9,5,6,9,2,4,3,9,3,3};
        int longitud=sizeof(ventasSemanales)/sizeof(ventasSemanales[0]);
        int menor = semanaMenorVent(ventasSemanales,longitud);
        printf("Longitud: %i\n",longitud);
        printf("la semana de menores ventas fue la numero: %d\n", menor+1);
        return 0;
    }