Performance Refactors

[Casperento]

Closes #34

This pull request refactors the PathExpr representation to improve performance, clarity, and maintainability. The core change replaces the std::variant and std::shared_ptr-based implementation with a simpler, custom pointer-based PathExpr class.

This new approach simplifies the data structure, reduces dynamic memory management overhead, and makes the path expression logic more explicit.

Major Changes

• Refactored PathExpr to a Custom Class: Replaced the std::variant-based PathExpr with a single, pointer-based class. This new class uses an enum for type dispatching instead of std::visit, eliminating dynamic polymorphism and simplifying memory management. All related creation and manipulation methods (createEdgeExpr, mergePaths, etc.) have been updated.

  • (Files: include/PHIGateAnalyzer.h, lib/PHIGateAnalyzer.cpp)

• Improved Performance and Determinism:

  • Optimization: Predicates are now computed only once per analyzed function, avoiding redundant work.
  • Determinism: Replaced the non-deterministic (random number-based) naming for outlined functions with a stable, sequential counter.

API Updates and Cleanup

• API Changes: Updated the ProgramSlice constructor and outline method signatures to reflect the new PathExpr interface.
  • (File: include/ProgramSlice.h)
• Code Simplification: Removed now-unnecessary helper structs and std::visit patterns. All traversal, collection, and printing logic has been consolidated into the new PathExpr class.
  • (Files: include/PHIGateAnalyzer.h, lib/PHIGateAnalyzer.cpp)