Skip to content

fix(rustlantis): drop the removed box_patterns feature gate - #1262

Merged
nihalpasham merged 1 commit into
NVIDIA:mainfrom
sachinsharma3191:fix/rustlantis-box-patterns
Sep 18, 2026
Merged

nihalpasham merged 1 commit into
NVIDIA:mainfrom
sachinsharma3191:fix/rustlantis-box-patterns

Conversation

@sachinsharma3191

@sachinsharma3191 sachinsharma3191 commented Sep 13, 2026 •

Copy link
Copy Markdown
Contributor

What this fixes

The Rustlantis generator builds again on the pinned nightly-2026-08-28. That nightly removed box_patterns; the vendored MIR crate still enabled it even though it contains no box patterns.

before: cargo build -p generate -> E0557: feature has been removed
after:  generator builds; mir_generator.py emits the seed-1 case

What changed

Removed the unused feature gate. I rebased and signed the contributor commit, preserving its author and sign-off.

Verification

Checked at eea15821:

  • Reinstating the gate reproduces E0557; removing it restores the build.
  • All 24 MIR/generator tests pass, formatting is clean, and the real Python adapter generates seed 1.
  • Hosted CI also passes on this head.

Strict Clippy in this vendored workspace still stops in unchanged abi/src/size.rs on manual_div_ceil. This change does not resolve that existing lint debt.

Related to the toolchain update in #1201. #1263 shares this prerequisite commit.

nightly-2026-08-28, pinned in NVIDIA#1201, removed the `box_patterns` feature,
so the vendored `mir` crate fails with E0557 and neither `generate` nor
`tools/mir_generator.py` builds. Nothing in the crate uses box patterns.
The rustlantis workspace is not built in CI, so the break went unnoticed.

Signed-off-by: sachin <sachinsharma3191@gmail.com>
@nihalpasham
nihalpasham force-pushed the fix/rustlantis-box-patterns branch from 86baab7 to eea1582 Compare September 18, 2026 03:48
@nihalpasham nihalpasham added bug Something isn't working build-related Build scripts, toolchain discovery, or compilation setup fuzzer Fuzzing infrastructure (rustlantis, difftest) labels Sep 18, 2026

@nihalpasham nihalpasham left a comment •

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I kept the one-line build fix and rebased and signed the contributor commit.

Restoring the removed feature gate reproduces E0557 on the pinned nightly. With it removed, all 24 MIR/generator tests pass and the Python adapter generates seed 1. Strict Clippy still stops on the existing abi/src/size.rs lint.

This restores the generator after the toolchain update in #1201 and supplies the prerequisite for #1263.

Checked at eea15821. Hosted CI also passes on this head.

@nihalpasham
nihalpasham merged commit cba1f5b into NVIDIA:main Sep 18, 2026
54 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working build-related Build scripts, toolchain discovery, or compilation setup fuzzer Fuzzing infrastructure (rustlantis, difftest)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants