Skip to content

Conversation

@kripken
Copy link
Member

@kripken kripken commented Sep 13, 2023

(tuple.extract 1
  (tuple.make (A) (B) (C))

=>

(B)

Modify some existing tests to not be in this trivial form, so that they do not
stop testing what they should.

@kripken kripken requested a review from tlively September 13, 2023 23:18
@codecov
Copy link

codecov bot commented Sep 13, 2023

Codecov Report

Merging #5938 (598aaa9) into main (9d79632) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #5938      +/-   ##
==========================================
+ Coverage   42.60%   42.61%   +0.01%     
==========================================
  Files         484      484              
  Lines       74815    74822       +7     
  Branches    11919    11920       +1     
==========================================
+ Hits        31872    31883      +11     
+ Misses      39746    39745       -1     
+ Partials     3197     3194       -3     
Files Changed Coverage Δ
src/passes/OptimizeInstructions.cpp 91.95% <100.00%> (+0.03%) ⬆️

... and 5 files with indirect coverage changes

@kripken kripken merged commit 183d4b0 into main Sep 14, 2023
@kripken kripken deleted the tuple.make.extract branch September 14, 2023 14:30
radekdoulik pushed a commit to dotnet/binaryen that referenced this pull request Jul 12, 2024
…ly#5938)

E.g.

(tuple.extract 1
  (tuple.make (A) (B) (C))

=>

(B)

Modify some existing tests to not be in this trivial form, so that they do not
stop testing what they should.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants