Static strings can be shared among multiple threads. Thus, we cannot register static StringImpl (StringImpl::isStatic() => true) to AtomicStringTable. When registering it, we newly allocate a StringImpl and register it to the table. Currently, everytime we register it, we allocate a new storage for underlying string. That is not good. We should create such a AtomicStringImpl owning original static string.
Ah, no. It is already done.