Skip to content

Commit 623da98

Browse files
sbc100sbc100
committed
Some fixes for library_webgl.js in CAN_ADDRESS_GB mode
This is an overall size win too after compression.
1 parent f7651be commit 623da98

13 files changed

+56
-41
lines changed

‎.circleci/config.yml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -798,8 +798,12 @@ jobs:
798798
steps:
799799
- run-tests-chrome:
800800
title: "browser"
801-
# skip test_4gb_fail as it OOMs on the current bot
802-
test_targets: "browser skip:browser.test_4gb_fail"
801+
# Skip test_4gb_fail as it OOMs on the current bot
802+
# Include an representative selection of browser2gb
803+
test_targets: "
804+
browser skip:browser.test_4gb_fail
805+
browser2gb.test_gles2_uniform_arrays
806+
"
803807
test-browser-chrome-wasm64:
804808
executor: bionic
805809
steps:

‎src/library_webgl.js

Lines changed: 16 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,12 @@ for (/**@suppress{duplicate}*/var i = 0; i < {{{ GL_POOL_TEMP_BUFFERS_SIZE }}};
8282
return HEAPU16;
8383
},
8484

85-
$heapAccessShiftForWebGLHeap: (heap) => 31 - Math.clz32(heap.BYTES_PER_ELEMENT),
85+
$toTypedArrayIndex: (pointer, heap) =>
86+
#if MEMORY64
87+
pointer / heap.BYTES_PER_ELEMENT,
88+
#else
89+
pointer >>> (31 - Math.clz32(heap.BYTES_PER_ELEMENT)),
90+
#endif
8691

8792
#if MIN_WEBGL_VERSION == 1
8893
$webgl_enable_ANGLE_instanced_arrays: (ctx) => {
@@ -1595,22 +1600,20 @@ for (/**@suppress{duplicate}*/var i = 0; i < {{{ GL_POOL_TEMP_BUFFERS_SIZE }}};
15951600
return colorChannels[format - 0x1902]||1;
15961601
},
15971602

1598-
$emscriptenWebGLGetTexPixelData__deps: ['$computeUnpackAlignedImageSize', '$colorChannelsInGlTextureFormat', '$heapObjectForWebGLType', '$heapAccessShiftForWebGLHeap'],
1603+
$emscriptenWebGLGetTexPixelData__deps: ['$computeUnpackAlignedImageSize', '$colorChannelsInGlTextureFormat', '$heapObjectForWebGLType', '$toTypedArrayIndex'],
15991604
$emscriptenWebGLGetTexPixelData: (type, format, width, height, pixels, internalFormat) => {
16001605
var heap = heapObjectForWebGLType(type);
1601-
var shift = heapAccessShiftForWebGLHeap(heap);
1602-
var byteSize = 1<<shift;
1603-
var sizePerPixel = colorChannelsInGlTextureFormat(format) * byteSize;
1606+
var sizePerPixel = colorChannelsInGlTextureFormat(format) * heap.BYTES_PER_ELEMENT;
16041607
var bytes = computeUnpackAlignedImageSize(width, height, sizePerPixel, GL.unpackAlignment);
16051608
#if GL_ASSERTIONS
1606-
assert((pixels >> shift) << shift == pixels, 'Pointer to texture data passed to texture get function must be aligned to the byte size of the pixel type!');
1609+
assert(pixels % heap.BYTES_PER_ELEMENT == 0, 'Pointer to texture data passed to texture get function must be aligned to the byte size of the pixel type!');
16071610
#endif
1608-
return heap.subarray(pixels >> shift, pixels + bytes >> shift);
1611+
return heap.subarray(toTypedArrayIndex(pixels, heap), toTypedArrayIndex(pixels + bytes, heap));
16091612
},
16101613

16111614
glTexImage2D__deps: ['$emscriptenWebGLGetTexPixelData'
16121615
#if MAX_WEBGL_VERSION >= 2
1613-
, '$heapObjectForWebGLType', '$heapAccessShiftForWebGLHeap'
1616+
, '$heapObjectForWebGLType', '$toTypedArrayIndex'
16141617
#endif
16151618
],
16161619
glTexImage2D: (target, level, internalFormat, width, height, border, format, type, pixels) => {
@@ -1641,7 +1644,7 @@ for (/**@suppress{duplicate}*/var i = 0; i < {{{ GL_POOL_TEMP_BUFFERS_SIZE }}};
16411644
GLctx.texImage2D(target, level, internalFormat, width, height, border, format, type, pixels);
16421645
} else if (pixels) {
16431646
var heap = heapObjectForWebGLType(type);
1644-
GLctx.texImage2D(target, level, internalFormat, width, height, border, format, type, heap, pixels >> heapAccessShiftForWebGLHeap(heap));
1647+
GLctx.texImage2D(target, level, internalFormat, width, height, border, format, type, heap, toTypedArrayIndex(pixels, heap));
16451648
} else {
16461649
GLctx.texImage2D(target, level, internalFormat, width, height, border, format, type, null);
16471650
}
@@ -1653,7 +1656,7 @@ for (/**@suppress{duplicate}*/var i = 0; i < {{{ GL_POOL_TEMP_BUFFERS_SIZE }}};
16531656

16541657
glTexSubImage2D__deps: ['$emscriptenWebGLGetTexPixelData'
16551658
#if MAX_WEBGL_VERSION >= 2
1656-
, '$heapObjectForWebGLType', '$heapAccessShiftForWebGLHeap'
1659+
, '$heapObjectForWebGLType', '$toTypedArrayIndex'
16571660
#endif
16581661
],
16591662
glTexSubImage2D: (target, level, xoffset, yoffset, width, height, format, type, pixels) => {
@@ -1674,7 +1677,7 @@ for (/**@suppress{duplicate}*/var i = 0; i < {{{ GL_POOL_TEMP_BUFFERS_SIZE }}};
16741677
GLctx.texSubImage2D(target, level, xoffset, yoffset, width, height, format, type, pixels);
16751678
} else if (pixels) {
16761679
var heap = heapObjectForWebGLType(type);
1677-
GLctx.texSubImage2D(target, level, xoffset, yoffset, width, height, format, type, heap, pixels >> heapAccessShiftForWebGLHeap(heap));
1680+
GLctx.texSubImage2D(target, level, xoffset, yoffset, width, height, format, type, heap, toTypedArrayIndex(pixels, heap));
16781681
} else {
16791682
GLctx.texSubImage2D(target, level, xoffset, yoffset, width, height, format, type, null);
16801683
}
@@ -1688,7 +1691,7 @@ for (/**@suppress{duplicate}*/var i = 0; i < {{{ GL_POOL_TEMP_BUFFERS_SIZE }}};
16881691

16891692
glReadPixels__deps: ['$emscriptenWebGLGetTexPixelData'
16901693
#if MAX_WEBGL_VERSION >= 2
1691-
, '$heapObjectForWebGLType', '$heapAccessShiftForWebGLHeap'
1694+
, '$heapObjectForWebGLType', '$toTypedArrayIndex'
16921695
#endif
16931696
],
16941697
glReadPixels: (x, y, width, height, format, type, pixels) => {
@@ -1700,7 +1703,7 @@ for (/**@suppress{duplicate}*/var i = 0; i < {{{ GL_POOL_TEMP_BUFFERS_SIZE }}};
17001703
GLctx.readPixels(x, y, width, height, format, type, pixels);
17011704
} else {
17021705
var heap = heapObjectForWebGLType(type);
1703-
GLctx.readPixels(x, y, width, height, format, type, heap, pixels >> heapAccessShiftForWebGLHeap(heap));
1706+
GLctx.readPixels(x, y, width, height, format, type, heap, toTypedArrayIndex(pixels, heap));
17041707
}
17051708
return;
17061709
}

‎src/library_webgl2.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -147,25 +147,25 @@ var LibraryWebGL2 = {
147147
GLctx.invalidateSubFramebuffer(target, list, x, y, width, height);
148148
},
149149

150-
glTexImage3D__deps: ['$heapObjectForWebGLType', '$heapAccessShiftForWebGLHeap'],
150+
glTexImage3D__deps: ['$heapObjectForWebGLType', '$toTypedArrayIndex'],
151151
glTexImage3D: (target, level, internalFormat, width, height, depth, border, format, type, pixels) => {
152152
if (GLctx.currentPixelUnpackBufferBinding) {
153153
GLctx.texImage3D(target, level, internalFormat, width, height, depth, border, format, type, pixels);
154154
} else if (pixels) {
155155
var heap = heapObjectForWebGLType(type);
156-
GLctx.texImage3D(target, level, internalFormat, width, height, depth, border, format, type, heap, pixels >> heapAccessShiftForWebGLHeap(heap));
156+
GLctx.texImage3D(target, level, internalFormat, width, height, depth, border, format, type, heap, toTypedArrayIndex(pixels, heap));
157157
} else {
158158
GLctx.texImage3D(target, level, internalFormat, width, height, depth, border, format, type, null);
159159
}
160160
},
161161

162-
glTexSubImage3D__deps: ['$heapObjectForWebGLType', '$heapAccessShiftForWebGLHeap'],
162+
glTexSubImage3D__deps: ['$heapObjectForWebGLType', '$toTypedArrayIndex'],
163163
glTexSubImage3D: (target, level, xoffset, yoffset, zoffset, width, height, depth, format, type, pixels) => {
164164
if (GLctx.currentPixelUnpackBufferBinding) {
165165
GLctx.texSubImage3D(target, level, xoffset, yoffset, zoffset, width, height, depth, format, type, pixels);
166166
} else if (pixels) {
167167
var heap = heapObjectForWebGLType(type);
168-
GLctx.texSubImage3D(target, level, xoffset, yoffset, zoffset, width, height, depth, format, type, heap, pixels >> heapAccessShiftForWebGLHeap(heap));
168+
GLctx.texSubImage3D(target, level, xoffset, yoffset, zoffset, width, height, depth, format, type, heap, toTypedArrayIndex(pixels, heap));
169169
} else {
170170
GLctx.texSubImage3D(target, level, xoffset, yoffset, zoffset, width, height, depth, format, type, null);
171171
}

‎test/code_size/hello_webgl2_wasm.json

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
{
22
"a.html": 569,
33
"a.html.gz": 379,
4-
"a.js": 4540,
5-
"a.js.gz": 2345,
4+
"a.js": 4589,
5+
"a.js.gz": 2341,
66
"a.wasm": 10451,
77
"a.wasm.gz": 6724,
8-
"total": 15560,
9-
"total_gz": 9448
8+
"total": 15609,
9+
"total_gz": 9444
1010
}

‎test/code_size/hello_webgl2_wasm2js.json

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
{
22
"a.html": 567,
33
"a.html.gz": 379,
4-
"a.js": 17746,
5-
"a.js.gz": 7982,
4+
"a.js": 17795,
5+
"a.js.gz": 7978,
66
"a.mem": 3123,
77
"a.mem.gz": 2693,
8-
"total": 21436,
9-
"total_gz": 11054
8+
"total": 21485,
9+
"total_gz": 11050
1010
}

‎test/code_size/hello_webgl_wasm.json

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
{
22
"a.html": 569,
33
"a.html.gz": 379,
4-
"a.js": 4027,
5-
"a.js.gz": 2173,
4+
"a.js": 4075,
5+
"a.js.gz": 2170,
66
"a.wasm": 10451,
77
"a.wasm.gz": 6724,
8-
"total": 15047,
9-
"total_gz": 9276
8+
"total": 15095,
9+
"total_gz": 9273
1010
}

‎test/code_size/hello_webgl_wasm2js.json

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
{
22
"a.html": 567,
33
"a.html.gz": 379,
4-
"a.js": 17219,
5-
"a.js.gz": 7814,
4+
"a.js": 17267,
5+
"a.js.gz": 7813,
66
"a.mem": 3123,
77
"a.mem.gz": 2693,
8-
"total": 20909,
9-
"total_gz": 10886
8+
"total": 20957,
9+
"total_gz": 10885
1010
}

‎test/other/metadce/test_metadce_hello_O0.gzsize

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
8408
1+
8403

‎test/other/metadce/test_metadce_hello_O0.jssize

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
23069
1+
23059

‎test/other/metadce/test_metadce_minimal_O0.gzsize

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
7238
1+
7232

0 commit comments

Comments
 (0)