09:00 | <Ashley Claymore> | I did report this Symbol.for stress test bug https://issues.chromium.org/issues/42202520 |
09:03 | <Ashley Claymore> | issue is open but patch looks approved so not sure if fixed or not |
09:04 | <Ashley Claymore> | looks like the patch was applied |
13:49 | <kriskowal> | This suggests to me that V8 holds symbols strongly, before and after this patch. Is that right? |
15:30 | <Ashley Claymore> | that is my understanding yes |
15:30 | <Ashley Claymore> | for registered symbols |
15:34 | <Ashley Claymore> | Compared to SpiderMonkey's: https://searchfox.org/mozilla-central/source/js/src/vm/SymbolType.h#146 where the map key is weak |
17:10 | <kriskowal> | Thank you, that’s helpful. |