diff options
author | Alon Zakai <azakai@google.com> | 2023-09-22 11:41:37 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-09-22 11:41:37 -0700 |
commit | a62ee0de8d0b1eccf8b5673b3ec9125cc44d3e3d (patch) | |
tree | d917c05319be58bcc9c25d1a81ccaff889683310 /test/gtest/stringify.cpp | |
parent | 7d4d502500bfd22a0a4e9c1762b1bddccbe14c1c (diff) | |
download | binaryen-a62ee0de8d0b1eccf8b5673b3ec9125cc44d3e3d.tar.gz binaryen-a62ee0de8d0b1eccf8b5673b3ec9125cc44d3e3d.tar.bz2 binaryen-a62ee0de8d0b1eccf8b5673b3ec9125cc44d3e3d.zip |
NameTypes and TypeSSA : Prefer _ over $ in names, and lint away _N suffixes (#5968)
Apparently $N (e.g. FooClass$5) is a convention in Java for anonymous classes, so our
$N that we use to disambiguate could be confusing. As the way we disambiguate does
not matter, switch to using _N. This PR does that in both TypeSSA and NameTypes.
Also make NameTypes "lint" names as it goes. That pass tries to give types nice names,
leaving existing ones that seem ok, and renaming long or unnamed ones. This PR makes
it aware of the _N notation and it tries to remove it, if removing it does not cause a
collision. An example of how that helps is if TypeSSA creates a subtype $Foo_0 and then
we manage to remove $Foo, then we can use the shorter name for the subtype.
Diffstat (limited to 'test/gtest/stringify.cpp')
0 files changed, 0 insertions, 0 deletions