Fix techscript_llvm_backend build failures and exhaustive match errors - #25
Conversation
- Migrated `LLVMPassManagerBuilder` to `LLVMPassBuilderOptions` as legacy PM was removed in LLVM 18. - Resolved exhaustive AST literal match errors in `codegen.rs`. - Handled missing terminator (Throw) and missing instruction (Try, EndTry, MakeDslBlock) AST match errors in codegen. - Fixed `ValueId` usage and type mismatch errors in context scopes (get_global vs get_local). - Upgraded JIT bindings to newer Orc2 APIs (`LLVMOrcCreateNewThreadSafeContext`, `LLVMOrcCreateNewThreadSafeModule`, `LLVMOrcDisposeLLJIT`). - Cleaned up obsolete imports and unsafe block requirements across the crate. Co-authored-by: Tcode-Motion <188012755+Tcode-Motion@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
1 participant
Resolves multiple build breakages and warnings in the
techscript_llvm_backendpackage resulting from the LLVM 18.0.0 migration. Replaced deprecated PassManager components and implemented the new PassBuilder structure for object compilation. Additionally fixed various non-exhaustive matches across AST literals and bytecode variants.PR created automatically by Jules for task 10737392408395134552 started by @Tcode-Motion