WebAssembly: fix erroneous signature comment
authorjfbastien@apple.com <jfbastien@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 13 Jun 2017 22:45:59 +0000 (22:45 +0000)
committerjfbastien@apple.com <jfbastien@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 13 Jun 2017 22:45:59 +0000 (22:45 +0000)
https://bugs.webkit.org/show_bug.cgi?id=173334

Reviewed by Keith Miller.

* wasm/WasmSignature.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@218220 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/wasm/WasmSignature.h

index 26cf9f1..9562cc7 100644 (file)
@@ -1,3 +1,12 @@
+2017-06-13  JF Bastien  <jfbastien@apple.com>
+
+        WebAssembly: fix erroneous signature comment
+        https://bugs.webkit.org/show_bug.cgi?id=173334
+
+        Reviewed by Keith Miller.
+
+        * wasm/WasmSignature.h:
+
 2017-06-13  Michael Saboff  <msaboff@apple.com>
 
         Refactor AbsenceOfSetter to AbsenceOfSetEffects
index 8f470d8..e413b25 100644 (file)
@@ -156,8 +156,8 @@ template<> struct HashTraits<JSC::Wasm::SignatureHash> : SimpleClassHashTraits<J
 
 namespace JSC { namespace Wasm {
 
-// Signature information is held globally on VM to allow all signatures to be unique. This is required when wasm calls another wasm instance.
-// Note: signatures are never removed from VM because that would require accounting for all WebAssembly.Module and which signatures they use. The maximum number of signatures is bounded, and isn't worth the counting overhead. We could clear everything when we reach zero outstanding WebAssembly.Module. https://bugs.webkit.org/show_bug.cgi?id=166037
+// Signature information is held globally and shared by VMs to allow all signatures to be unique. This is required when wasm calls another wasm instance, and must work when modules are shared between multiple VMs.
+// Note: signatures are never removed because that would require accounting for all WebAssembly.Module and which signatures they use. The maximum number of signatures is bounded, and isn't worth the counting overhead. We could clear everything when we reach zero outstanding WebAssembly.Module. https://bugs.webkit.org/show_bug.cgi?id=166037
 class SignatureInformation {
     WTF_MAKE_NONCOPYABLE(SignatureInformation);