cucapra / dahlia-evaluation

Evaluation for Dahlia
3 stars 0 forks source link

Verify gemm-ncubed version for rewrite vs basline comparison #33

Closed rachitnigam closed 4 years ago

rachitnigam commented 4 years ago

Figure out which version of gemm-ncubed kernel ran for the rewrite vs baseline comparison. The changes committed here do not compile: https://github.com/cucapra/dahlia-evaluation/commit/d2b3fbfae16e96ea0d83e1e40745263144ee7315

sgpthomas commented 4 years ago

I'm pretty sure this is the kernel used for the rewrite (with real numbers of course). but I can't be sure unless we still have the Cerberus jobs still.

rachitnigam commented 4 years ago

Cerberus is up now. @sgpthomas can you verify this and close this issue?

sgpthomas commented 4 years ago

This was the kernel that was run

decl m1: double[64][64];
decl m2: double[64][64];
decl prod: double[64][64];

for (let i = 0..64) {
  for (let j = 0..64) {
    let sum: double = 0.0;
    for (let k = 0..64) {
      let mult = m1[i][k] * m2[k][j];
    } combine {
      sum += mult;
    }
    prod[i][j] := sum;
  }
}