Sam Clegg

Or we could simple remove the header completely.. that seems like it would also work for libc++

> `emcc -fprofile-instr-generate -fcoverage-mapping demo.cpp` > > wasm-ld: error: /var/folders/61/x35v93n50_5g1dfjrtprr2dw0000gn/T/emscripten_temp_xjrzk63_/demo_0.o: undefined symbol: __llvm_profile_runtime > > need help, emcc seems not support -fcoverage-mapping? No, emcc doesn't support this yet. This PR...

No update I'm afraid. Let is know if you manage to get stuff working.

Closing in favor of #13072

Assuming this works do we think its good idea? I wonder how much it would slow down tests with a lof stdio? Since browser tests are already run iserially it...

Looks like we raced to comment :)

It seems like at some point browser tests start failing due to lack a GL context.. its almost like the browser stops supported GL for all following tests.

> Does this not affect e.g. `./emcc test/hello_world.c -lxlib.js`? That works now but after this change it would need to be `./emcc test/hello_world.c -lX11.js` Yes, that is true. However, the...

Ok, I think this is ready to land now. Hopefully the tests will pass..

Seems like it could be related to the node version being out of date on emscripten-releases. emsdk is 14.18.2 and emscripten-releases is 14.15.5.