Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CFE] dart2js/mixin_super/main fuzzing failure #49339

Open
jensjoha opened this issue Jun 27, 2022 · 0 comments
Open

[CFE] dart2js/mixin_super/main fuzzing failure #49339

jensjoha opened this issue Jun 27, 2022 · 0 comments
Labels
area-front-end Use area-front-end for front end / CFE / kernel format related issues.

Comments

@jensjoha
Copy link
Contributor

Running these two:

out/ReleaseX64/dart --enable-asserts pkg/front_end/test/fasta/strong_suite.dart -DsemiFuzz=true -- strong/dart2js/mixin_super/...
out/ReleaseX64/dart --enable-asserts pkg/front_end/test/fasta/weak_suite.dart -DsemiFuzz=true -- weak/dart2js/mixin_super/...

gives e.g.:

Compiled and got 2 libs
 -> invalidating [...]/pkg/front_end/testcases/dart2js/mixin_super/main.dart
 -> and got 2 libs
Sets {_Class4b&Class4a&Mixin2.field1, _Class4b&Class4a&Mixin2.field2, _Class4b&Class4a&Mixin2.field2, _Class4b&Class4a&Mixin2.property2, _Class4b&Class4a&Mixin2.property2, _Class4b&Class4a&Mixin2.method2, _Class4b&Class4a&Mixin2.method1, _Class4b&Class4a&Mixin2.property1, _Class4b&Class4a&Mixin2.property1, _Class4b&Class4a&Mixin2.field1} and {_Class4b&Class4a&Mixin2.property2, _Class4b&Class4a&Mixin2.property2, _Class4b&Class4a&Mixin2.field1, _Class4b&Class4a&Mixin2.field2, _Class4b&Class4a&Mixin2.field2, _Class4b&Class4a&Mixin2._#Mixin2#field1#AI, _Class4b&Class4a&Mixin2._#Mixin2#field1#AI, _Class4b&Class4a&Mixin2._#Mixin2#field2#AI, _Class4b&Class4a&Mixin2._#Mixin2#field2#AI, _Class4b&Class4a&Mixin2.method2, _Class4b&Class4a&Mixin2.method1, _Class4b&Class4a&Mixin2.property1, _Class4b&Class4a&Mixin2.property1, _Class4b&Class4a&Mixin2.field1} are not equivalent
.root
 Library(library [...]/pkg/front_end/testcases/dart2js/mixin_super/main.dart).classes[12]
  Class(_Class4b&Class4a&Mixin2).procedures.length
Sets {_Class5b&Class5a&Mixin2.field1, _Class5b&Class5a&Mixin2.field2, _Class5b&Class5a&Mixin2.field2, _Class5b&Class5a&Mixin2.property2, _Class5b&Class5a&Mixin2.property2, _Class5b&Class5a&Mixin2.method2, _Class5b&Class5a&Mixin2.method1, _Class5b&Class5a&Mixin2.property1, _Class5b&Class5a&Mixin2.property1, _Class5b&Class5a&Mixin2.field1} and {_Class5b&Class5a&Mixin2.property2, _Class5b&Class5a&Mixin2.property2, _Class5b&Class5a&Mixin2.field1, _Class5b&Class5a&Mixin2.field2, _Class5b&Class5a&Mixin2.field2, _Class5b&Class5a&Mixin2._#Mixin2#field1#AI, _Class5b&Class5a&Mixin2._#Mixin2#field1#AI, _Class5b&Class5a&Mixin2._#Mixin2#field2#AI, _Class5b&Class5a&Mixin2._#Mixin2#field2#AI, _Class5b&Class5a&Mixin2.method2, _Class5b&Class5a&Mixin2.method1, _Class5b&Class5a&Mixin2.property1, _Class5b&Class5a&Mixin2.property1, _Class5b&Class5a&Mixin2.field1} are not equivalent
.root
 Library(library [...]/pkg/front_end/testcases/dart2js/mixin_super/main.dart).classes[15]
  Class(_Class5b&Class5a&Mixin2).procedures.length
Sets {_Class6b&Class6a&Mixin2.field1, _Class6b&Class6a&Mixin2.field2, _Class6b&Class6a&Mixin2.field2, _Class6b&Class6a&Mixin2.property2, _Class6b&Class6a&Mixin2.property2, _Class6b&Class6a&Mixin2.method2, _Class6b&Class6a&Mixin2.method1, _Class6b&Class6a&Mixin2.property1, _Class6b&Class6a&Mixin2.property1, _Class6b&Class6a&Mixin2.field1} and {_Class6b&Class6a&Mixin2.property2, _Class6b&Class6a&Mixin2.property2, _Class6b&Class6a&Mixin2.field1, _Class6b&Class6a&Mixin2.field2, _Class6b&Class6a&Mixin2.field2, _Class6b&Class6a&Mixin2._#Mixin2#field1#AI, _Class6b&Class6a&Mixin2._#Mixin2#field1#AI, _Class6b&Class6a&Mixin2._#Mixin2#field2#AI, _Class6b&Class6a&Mixin2._#Mixin2#field2#AI, _Class6b&Class6a&Mixin2.method2, _Class6b&Class6a&Mixin2.method1, _Class6b&Class6a&Mixin2.property1, _Class6b&Class6a&Mixin2.property1, _Class6b&Class6a&Mixin2.field1} are not equivalent
.root
 Library(library [...]/pkg/front_end/testcases/dart2js/mixin_super/main.dart).classes[18]
  Class(_Class6b&Class6a&Mixin2).procedures.length


Fuzzed component changed in an unexpected way.
1 failed:
strong/dart2js/mixin_super/main: SemiFuzzFailure

Where on the recompile seemingly there's suddenly these extra entries in a class (etc):

_Class4b&Class4a&Mixin2._#Mixin2#field1#AI
_Class4b&Class4a&Mixin2._#Mixin2#field1#AI
_Class4b&Class4a&Mixin2._#Mixin2#field2#AI
_Class4b&Class4a&Mixin2._#Mixin2#field2#AI

/cc @johnniwinther

@jensjoha jensjoha added the area-front-end Use area-front-end for front end / CFE / kernel format related issues. label Jun 27, 2022
copybara-service bot pushed a commit that referenced this issue Jun 27, 2022
Bug: #49339

Change-Id: I305a1099ce0c3863a24a7f21a66fdd67fbbee3a6
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/249802
Reviewed-by: Johnni Winther <johnniwinther@google.com>
Commit-Queue: Jens Johansen <jensj@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-front-end Use area-front-end for front end / CFE / kernel format related issues.
Projects
None yet
Development

No branches or pull requests

1 participant