-
-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Labels
enhancementNew feature or requestNew feature or requestphase-3Phase 3 of epic implementationPhase 3 of epic implementation
Description
Phase 3: Remove Hardcoded DtRobust System
Part of Epic #247
Objective
Remove all hardcoded DtRobust logic from the compiler IR, making it operator-agnostic and letting the standard library be authoritative.
Tasks
Remove Type Definitions
- Remove
DtRobustOperatorenum from types.rs - Remove
IntegrationMethodenum from types.rs - Remove
DtRobustCallvariant fromCompiledExpr - Remove
DtRobustCallfromSsaInstruction
Remove Hardcoded Parsing
- Remove
parse_dt_robust_operator()function from lower/expr.rs - Remove hardcoded dt-robust check in expression lowering
- Remove dt-robust precedence over kernel function lookup
- Update
dt_robust_operator_name()in codegen.rs (or remove entirely)
Remove Execution Logic
- Remove
execute_dt_robust()from vectorized/mod.rs - Remove dt-robust handling from SSA execution
- Remove pattern matching on DtRobust variants
Update Related Code
- Remove dt-robust imports across the codebase
- Update any documentation referencing hardcoded operators
- Remove or update related test files
Validation
- Ensure no compilation errors after removal
- Verify fallback to kernel function calls works
- Test with simple dt-robust function calls
Files to Modify
crates/kernels/ir/src/types.rscrates/kernels/ir/src/expressions.rscrates/kernels/ir/src/ssa/types.rscrates/kernels/ir/src/lower/expr.rscrates/kernels/ir/src/vectorized/mod.rscrates/kernels/ir/src/codegen.rs- Various test files
Success Criteria
- No hardcoded dt-robust logic in compiler
- Compiler builds successfully
- DtRobust functions route to stdlib instead
- All tests pass (updated for new behavior)
Dependencies
- Requires Phase 2 (complete stdlib implementation)
Breaking Changes
DtRobustOperatorand related types removed from public API- DSL must use
kernel.prefix for dt-robust functions
Estimated Time: 1 week
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or requestphase-3Phase 3 of epic implementationPhase 3 of epic implementation