Skip to content

Remove duplicate runtime IR files#9

Merged
tcrypt25519 merged 2 commits into
masterfrom
copilot/remove-duplicate-runtime-ir-file
Feb 3, 2026
Merged

Remove duplicate runtime IR files#9
tcrypt25519 merged 2 commits into
masterfrom
copilot/remove-duplicate-runtime-ir-file

Conversation

Copilot AI commented Feb 3, 2026

Copy link
Copy Markdown

Three identical copies of jet.ll existed in the repository. Consolidated to single source of truth at runtime-ir/jet.ll.

Changes

  • Removed duplicates:

    • crates/jet/runtime-ir/jet.ll
    • examples/jet.ll
  • Updated documentation:

    • docs/manifesto.md: Removed duplicate file references, clarified runtime-ir/jet.ll as source of truth

Context

Engine loads runtime IR via relative path "runtime-ir/jet.ll" from workspace root. This path continues to resolve correctly to the remaining file.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/repos/TheDan64/inkwell/commits/HEAD
    • Triggering command: /home/REDACTED/.rustup/toolchains/stable-x86_64-REDACTED-linux-gnu/bin/cargo /home/REDACTED/.rustup/toolchains/stable-x86_64-REDACTED-linux-gnu/bin/cargo metadata --format-version 1 (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>Remove or centralize duplicate runtime IR file to prevent drift</issue_title>
<issue_description>Pick a single source-of-truth for jet.ll (preferably runtime-ir/jet.ll), remove the unused duplicate at crates/jet/runtime-ir/jet.ll, and update any docs or build scripts that mention the duplicate. If a copy is required for packaging, add a build step or script to generate it from the source-of-truth file instead of manually duplicating.</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@semanticdiff-com

Copy link
Copy Markdown

Review changes with  SemanticDiff

Co-authored-by: tcrypt25519 <212655132+tcrypt25519@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove or centralize duplicate runtime IR file Remove duplicate runtime IR files Feb 3, 2026
Copilot AI requested a review from tcrypt25519 February 3, 2026 16:07
@tcrypt25519
tcrypt25519 marked this pull request as ready for review February 3, 2026 18:50
Copilot AI review requested due to automatic review settings February 3, 2026 18:50
@tcrypt25519
tcrypt25519 merged commit 4303257 into master Feb 3, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request consolidates three identical copies of the runtime LLVM IR file (jet.ll) to a single source of truth at the workspace root. The PR successfully removes the duplicate files and updates documentation to reflect this change.

Changes:

  • Removed duplicate jet.ll files from crates/jet/runtime-ir/ and examples/ directories
  • Updated docs/manifesto.md to document runtime-ir/jet.ll as the single source of truth for runtime module layout
  • Clarified build system documentation to remove references to duplicate files

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
examples/jet.ll Removed duplicate runtime IR file that was identical to the source of truth
crates/jet/runtime-ir/jet.ll Removed duplicate runtime IR file that was identical to the source of truth
docs/manifesto.md Updated documentation to remove duplicate references and clarify single source of truth at runtime-ir/jet.ll

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@tcrypt25519
tcrypt25519 deleted the copilot/remove-duplicate-runtime-ir-file branch February 10, 2026 12:06
tcrypt25519 added a commit that referenced this pull request Feb 16, 2026
…me-ir-file

Remove duplicate runtime IR files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove or centralize duplicate runtime IR file to prevent drift

3 participants