canonicalize specilize args to always use SpecializationArg::Struct/Enum

[TomerStarkware]

No description provided.

[reviewable-StarkWare]

This change is 

[TomerStarkware]

• canonicalize specilize args to always use SpecializationArg::Struct/Enum #9021 👈 (View in Graphite)
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[orizi]

@orizi reviewed 6 of 6 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @TomerStarkware)

---

crates/cairo-lang-lowering/src/optimizations/const_folding.rs line 60 at r1 (raw file):

        ConstValue::Struct(members, ty) => {
            // Only convert to SpecializationArg::Struct if the type is actually a concrete struct,
            // not a closure or fixed size array.

why not fixed-sized arrays and tuples as well?

[TomerStarkware]

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @orizi)

---

crates/cairo-lang-lowering/src/optimizations/const_folding.rs line 60 at r1 (raw file):

Previously, orizi wrote…

why not fixed-sized arrays and tuples as well?

Currently, we do not support partials of those anyway, and if I were to break them here, there is no code that's going to reconstruct them. We need to add specializationArg:{tuple,fixedsize, closure} for this to work.

[orizi]

Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @TomerStarkware)