diff options
| author | Thomas Lively <tlively@google.com> | 2024-08-06 14:25:04 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-08-06 18:25:04 +0000 |
| commit | d5a5425c0c76cfc08711b81d6ec70c3a2879e405 (patch) | |
| tree | cb8f2dae76598416e833c1818438a46bf7666e19 /src/tools/fuzzing | |
| parent | a985e16832f86cbc42eb6488e6f9fc2a9eff7dc7 (diff) | |
| download | binaryen-d5a5425c0c76cfc08711b81d6ec70c3a2879e405.tar.gz binaryen-d5a5425c0c76cfc08711b81d6ec70c3a2879e405.tar.bz2 binaryen-d5a5425c0c76cfc08711b81d6ec70c3a2879e405.zip | |
Restore isString type methods (#6815)
PR ##6803 proposed removing Type::isString and HeapType::isString in
favor of more explicit, verbose callsites. There was no consensus to
make this change, but it was accidentally committed as part of #6804.
Revert the accidental change, except for the useful, noncontroversial
parts, such as fixing the `isString` implementation and a few other
locations to correctly handle shared types.
Diffstat (limited to 'src/tools/fuzzing')
| -rw-r--r-- | src/tools/fuzzing/fuzzing.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/tools/fuzzing/fuzzing.cpp b/src/tools/fuzzing/fuzzing.cpp index 138831860..2aff7146e 100644 --- a/src/tools/fuzzing/fuzzing.cpp +++ b/src/tools/fuzzing/fuzzing.cpp @@ -2536,8 +2536,7 @@ Expression* TranslateToFuzzReader::makeConst(Type type) { if (type.isNullable() && oneIn(8)) { return builder.makeRefNull(type.getHeapType()); } - if (type.getHeapType().isMaybeShared(HeapType::string)) { - assert(!type.getHeapType().isShared() && "TODO: shared strings"); + if (type.getHeapType().isString()) { return makeStringConst(); } if (type.getHeapType().isBasic()) { |