CMake: Hide symbols from wasm engines included

[chfast]

For binaryen + wabt:

nm -Cg src/libhera.so | wc -l 
readelf -r src/libhera.so | wc -l

Before: 9852 10831
After: 3993 6510

[chfast]

I wander why tests on macos are failing...

[chfast]

@cdetrio Can you benchmark wabt engine built like this?

[axic]

There some exception happening:

Totally unknown exception
/Users/distiller/project/test/tools/libtesteth/ImportTest.cpp:768: error: in "GeneralStateTests/stEWASMTests": malformedBytecodeInvalidWasm on Byzantium: Expected another postState hash! expected: 0x0bf1306fe3199b9863a81ed7cb60154a9d94076356eec1fe7654f423999fbcdd actual: 0xf93bc41346c0dad509a9e1ede7e6348b91c9e786999144a44ab5533ecf95698b in Byza

Remove that catch statement in hera.cpp to see what's wrong.

[codecov-io]

Codecov Report

Merging #518 into master will increase coverage by 0.15%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #518      +/-   ##
==========================================
+ Coverage   52.18%   52.33%   +0.15%     
==========================================
  Files           8        8              
  Lines        1307     1307              
  Branches      130      130              
==========================================
+ Hits          682      684       +2     
+ Misses        596      594       -2     
  Partials       29       29

[axic]

Executing message in Hera
libc++abi.dylib: terminating with uncaught exception of type wasm::ParseException
unknown location:0: fatal error: in "GeneralStateTests/stEWASMTests": signal: SIGABRT (application abort requested)
/Users/distiller/project/test/tools/libtesteth/TestHelper.cpp:617: last checkpoint

Somewhere the parser exception is not caught in binaryen. BinaryenEngine::loadModule is the place it should be usually handled.

[chfast]

Somewhere the parser exception is not caught in binaryen. BinaryenEngine::loadModule is the place it should be usually handled.

Ok, but why it happened now?

[axic]

Ok, but why it happened now?

The only thing I could imagine is that some symbols are runtime resolved and by hiding them that is not possible anymore.

[chfast]

Removed the change for Binaryen