Return an Array from a function C: Subscripted value is not an array, pointer, or vector

Question

I need to return the value of the matrix, but I am gettin this error

Subscripted value is not an array, pointer, or vector

in:

qk_output[m][o] = 0;

and

qk_output[m][o] += queries[m][n] * keys[n][o];

Could anyone help me? Thanks!

int* multmm(int queries[M][N], int keys[N][O]) {
  // Matrix Multiplication
  int* qk_output = (int*)malloc(sizeof(int) *M*N);
  for (int m = 0; m < M; m++) {

    for (int o = 0; o < O; o ++) {
      qk_output[m][o] = 0;

      for (int n = 0; n < N; n++) {
        qk_output[m][o] += queries[m][n] * keys[n][o];
      }
    }
  }
 return qk_output;
}

qk_output is not a 2D array. qk_output is an int*. Thus qk_output[m] is an int, and qk_output[m][o] is invalid. — n. m. could be an AI
– n. m. could be an AI, Commented Oct 30, 2021 at 5:48
If you are going to use a flat 1D array and simulate a 2D array, you need to change qk_output[m][o] to qk_output[m * N + o] — David C. Rankin
– David C. Rankin, Commented Oct 30, 2021 at 6:18

4386427 · Accepted Answer · 2021-10-30 06:26:16Z

1

To return a 2D array from a function, you can do

#define N 4
#define M 3

int (*foo(void ))[N]
{
    int (*p)[N] = malloc(M * sizeof *p);
    for (int i = 0; i < M; ++i)
    {
        for (int j = 0; j < N; ++j)
        {
            p[i][j] = i + j;
        }
    }
    return p;
}

int main (void){
    int (*mat)[N] = foo();
    
    for (int i = 0; i < M; ++i)
    {
        for (int j = 0; j < N; ++j)
        {
            printf("%d ", mat[i][j]);
        }
        puts("");
    }
    
    free(mat);
    return 0;   
}

answered Oct 30, 2021 at 6:26

community wiki

4386427

Sign up to request clarification or add additional context in comments.

Comments

David Ranieri · Accepted Answer · 2021-10-30 06:53:15Z

1

int* qk_output = (int*)malloc(sizeof(int) *M*N);

qk_output is a pointer to an int

the compiler knows how to access qk_output[n] but doesn't know how to access qk_output[m][n], you need to switch to:

int (*qk_output)[N] = malloc(sizeof(*qk_output) * M); // Don't cast malloc

that is, a pointer to an array of N ints

Now the compiler has enough information to access qk_output[m][n]

For the return question: you can use void *multmm(...) or int (*multmm(...))[N], the second one gives more information to the compiler so it is less error prone.

Something like:

int (*multmm(int queries[M][N], int keys[N][O]))[N]
{
    int (*qk_output)[N] = malloc(sizeof(*qk_output) * M);
    ...
    return qk_output;
}

int main(void)
{
    int (*arr2D)[N] = multmm(...);
    ...
}

edited Oct 30, 2021 at 6:53

answered Oct 30, 2021 at 5:49

David Ranieri

41.2k8 gold badges58 silver badges103 bronze badges

2 Comments

mmmtt Over a year ago

Thank you very much but, I still have a similar error, in the malloc line copying the same code you mentioned here. Error: ERROR: [HLS 207-2754] cannot initialize a variable of type 'int (*)[2]' with an rvalue of type 'void *' (I have tried to do the casting in the malloc also and I get the same error)

David Ranieri Over a year ago

It should workd: ideone.com/VlDTzy Try to extract a short fragment and it will be easier to help.

Collectives™ on Stack Overflow

Return an Array from a function C: Subscripted value is not an array, pointer, or vector

2 Answers 2

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related