Added shared memory support in binaryen definition

[nidin]

Extended _BinaryenAddMemoryImport and _BinaryenSetMemory functions with shared: bool argument
This will configure WASM memory as shared.

[dcodeIO]

As a starting point it is ok to just append a , false where these functions are called, just to make the tests pass. Also please append yourself to the NOTICE file before I merge :)

[MaxGraey]

For declarations I guess last param should be define as shared?: bool

[nidin]

It’s a C API. there is no optional arguments nor default value for the argument, therefore it’s a breaking change.

…

On Thu 24. Jan 2019 at 4:52 PM, Max Graey ***@***.***> wrote: For declarations I guess last param should be define as shared?: bool — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#435 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAMGZtqUQt2Js2OXc_c9KjSnWe8nbWVcks5vGdbBgaJpZM4aQup7> .

[nidin]

May be JS API of binaryen will take it as false in the absence of this argument.

…

On Thu 24. Jan 2019 at 8:27 PM, 01Alchemist ***@***.***> wrote: It’s a C API. there is no optional arguments nor default value for the argument, therefore it’s a breaking change. On Thu 24. Jan 2019 at 4:52 PM, Max Graey ***@***.***> wrote: > For declarations I guess last param should be define as shared?: bool > > — > You are receiving this because you authored the thread. > Reply to this email directly, view it on GitHub > <#435 (comment)>, > or mute the thread > <https://github.com/notifications/unsubscribe-auth/AAMGZtqUQt2Js2OXc_c9KjSnWe8nbWVcks5vGdbBgaJpZM4aQup7> > . >

[MaxGraey]

I see. In this case need add additional and optional shared param for this methods:

1. assemblyscript/src/module.ts

   Line 905 in d5f72e3

   addMemoryImport(

2. assemblyscript/src/module.ts

   Line 943 in d5f72e3

   setMemory(

[nidin]

@dcodeIO @MaxGraey Ready for review

[dcodeIO]

Looking good, merging. Thanks! :)