Apply ReturnOptimization to the input of the match optimizer test.

[ilyalesokhin-starkware]

No description provided.

[reviewable-StarkWare]

This change is 

[ilyalesokhin-starkware]

crates/cairo-lang-lowering/src/optimizations/test_data/match_optimization line 768 at r1 (raw file):

  (v7: test::Color) <- Color::Red(v2)
End:
  Return(v6, v7)

looks like this test is broken :(

Code quote:

Return(v6, v7)

[ilyalesokhin-starkware]

crates/cairo-lang-lowering/src/optimizations/test_data/match_optimization line 1656 at r1 (raw file):

//! > test_comments
// TODO(orizi): Enable the optimization for this case as well.

it is applicable when we apply the return optimization

Code quote:

// TODO(orizi): Enable the optimization for this case as well.

[ilyalesokhin-starkware]

crates/cairo-lang-lowering/src/optimizations/test_data/match_optimization line 768 at r1 (raw file):

Previously, ilyalesokhin-starkware wrote…

looks like this test is broken :(

fixed

[orizi]

Reviewed 1 of 2 files at r1, all commit messages.
Reviewable status: 1 of 2 files reviewed, 3 unresolved discussions (waiting on @ilyalesokhin-starkware)

---

crates/cairo-lang-lowering/src/optimizations/test_data/match_optimization line 1656 at r1 (raw file):

Previously, ilyalesokhin-starkware wrote…

it is applicable when we apply the return optimization

i don't understand.

---

a discussion (no related file):
is this preferable? note that it "hide" part of what the optimization actually did.
maybe add it as a boolean for the test params?

[ilyalesokhin-starkware]

Previously, orizi wrote…

is this preferable? note that it "hide" part of what the optimization actually did.
maybe add it as a boolean for the test params?

what does it hide?

[orizi]

Reviewed 1 of 1 files at r2.
Reviewable status: all files reviewed, 3 unresolved discussions (waiting on @ilyalesokhin-starkware)

---

a discussion (no related file):

Previously, ilyalesokhin-starkware wrote…

what does it hide?

the name and the block ids change further, and remappings are deleted.

[ilyalesokhin-starkware]

Previously, orizi wrote…

the name and the block ids change further, and remappings are deleted.

in the input to the optimization, not in the result.

[orizi]

Reviewable status: all files reviewed, 1 unresolved discussion

---

a discussion (no related file):

Previously, ilyalesokhin-starkware wrote…

in the input to the optimization, not in the result.

oh - nm.

[orizi]

Reviewed 1 of 1 files at r3, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion