Change function identifier in profiling to fully qualified path

[piotmag769]

This change is 

[orizi]

Reviewed 4 of 5 files at r1, all commit messages.
Reviewable status: 4 of 5 files reviewed, 1 unresolved discussion (waiting on @piotmag769)

---

crates/cairo-lang-runner/src/profiling_test_data/major_test_cases line 75 at r1 (raw file):

  function test::pow2_14000: 12
Weight by Cairo function:
  function unknown: 126021

seems there's some mishandling there.

Code quote:

  function unknown: 126021

[orizi]

Reviewed 4 of 5 files at r2, all commit messages.
Reviewable status: 5 of 6 files reviewed, 1 unresolved discussion (waiting on @piotmag769)

---

crates/cairo-lang-semantic/src/test_utils.rs line 124 at r2 (raw file):

    let x = PathBuf::from("lib.cairo");
    std::fs::write(&x, content.as_bytes()).unwrap();
    let file_id = db.intern_file(FileLongId::OnDisk(x));

revert this.
just discover in the print case due to the missing Parent in the virtual file.
this may cause weird test collisions.

Code quote:

    let x = PathBuf::from("lib.cairo");
    std::fs::write(&x, content.as_bytes()).unwrap();
    let file_id = db.intern_file(FileLongId::OnDisk(x));

[orizi]

Reviewed 1 of 5 files at r2, 1 of 1 files at r3, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @piotmag769)

[piotmag769]

Reviewed 1 of 5 files at r1, 2 of 5 files at r2, 1 of 1 files at r3, 2 of 2 files at r4, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @orizi)

---

crates/cairo-lang-sierra-generator/src/statements_locations.rs line 28 at r4 (raw file):

            let file_name = file_id.file_name(db.upcast());

            while let FileLongId::Virtual(VirtualFile { parent: Some(parent), .. }) =

Will split this function to make sure the cases where:

• parent for VirtualFile is missing
• the ModuleId for real file is missing

are discoverable from the outside on the type level. These cases shouldn't happen but better to be safe than sorry.

---

crates/cairo-lang-semantic/src/test_utils.rs line 124 at r2 (raw file):

Previously, orizi wrote…

revert this.
just discover in the print case due to the missing Parent in the virtual file.
this may cause weird test collisions.

Accidentally committed

[orizi]

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

[piotmag769]

Reviewed 2 of 2 files at r5, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @piotmag769)

---

crates/cairo-lang-sierra-generator/src/statements_locations.rs line 28 at r4 (raw file):

Previously, piotmag769 (Piotr Magiera) wrote…

Will split this function to make sure the cases where:

• parent for VirtualFile is missing
• the ModuleId for real file is missing

are discoverable from the outside on the type level. These cases shouldn't happen but better to be safe than sorry.

Done