Commit 4792e66
committed
[ET-VK] Fixing out_limits_scaled calculation for conv2d pw ops.
The fix changing the calculation of `out_limits_scaled` from:
```glsl
const int out_limits_scaled[2] = {out_limits.x + (TILE_SIZE_X - 1) * TILE_SIZE_X, out_limits.y + (TILE_SIZE_Y - 1) * TILE_SIZE_Y};
```
to:
```glsl
const int out_limits_scaled[2] = {(out_limits.x + (TILE_SIZE_X - 1)) / TILE_SIZE_X, (out_limits.y + (TILE_SIZE_Y - 1)) / TILE_SIZE_Y};
```
This change ensures that `out_limits_scaled` is calculated correctly, taking into account the tile size and the output limits of the convolution operation.
Differential Revision: [D75575662](https://our.internmc.facebook.com/intern/diff/D75575662/)
[ghstack-poisoned]1 parent 95e883c commit 4792e66
File tree
2 files changed
+2
-2
lines changed- backends/vulkan/runtime/graph/ops/glsl
2 files changed
+2
-2
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
46 | 46 | | |
47 | 47 | | |
48 | 48 | | |
49 | | - | |
| 49 | + | |
50 | 50 | | |
51 | 51 | | |
52 | 52 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
48 | 48 | | |
49 | 49 | | |
50 | 50 | | |
51 | | - | |
| 51 | + | |
52 | 52 | | |
53 | 53 | | |
54 | 54 | | |
| |||
0 commit comments