BE-268: HashQL: Rename PreInlining to PreInline for consistency

[indietyp]

🌟 What is the purpose of this PR?

Rename the PreInlining pass to PreInline for consistency with other transform passes in the codebase. This includes renaming the associated files, module references, and test directories.

🔍 What does this change?

• Renamed PreInlining class to PreInline in the MIR transform passes
• Renamed pre_inlining.rs file to pre_inline.rs
• Updated all import references to use the new name
• Moved test files from pre_inlining directory to pre_inline
• Updated suite name from "mir/pass/transform/pre-inlining" to "mir/pass/transform/pre-inline"
• Updated all function names and references from mir_pass_transform_pre_inlining to mir_pass_transform_pre_inline

Pre-Merge Checklist 🚀

🚢 Has this modified a publishable library?

This PR:

• does not modify any publishable blocks or libraries, or modifications do not need publishing

📜 Does this require a change to the docs?

The changes in this PR:

• are internal and do not require a docs change

🕸️ Does this require a change to the Turbo Graph?

The changes in this PR:

• do not affect the execution graph

🛡 What tests cover this?

Existing tests were moved to the new directory structure and continue to cover the functionality.

❓ How to test this?

1. Run the existing test suite to verify that the renamed components work correctly
2. Verify that all imports and references to the renamed components are working properly

[cursor]

PR Summary

Aligns naming of the MIR pre-inlining pass across the codebase.

• Renames transform::PreInlining to transform::PreInline, updating exports in transform/mod.rs and implementing the new pre_inline.rs
• Updates compiletest suites to mir_pass_transform_pre_inline and suite name "mir/pass/transform/pre-inline"; adjusts callers (e.g., mir_pass_transform_inline)
• Replaces all imports/usages (PreInlining → PreInline) in benches and compiletest
• Moves/renames tests from tests/ui/pass/pre_inlining to tests/ui/pass/pre_inline with updated .spec.toml and outputs

^{Written by Cursor Bugbot for commit 447336b. This will update automatically on new commits. Configure here.}

[indietyp]

• BE-304: HashQL: Terminator placement based on capabilities #8359
• BE-303: HashQL: Split MIR basic blocks by target support #8354
• BE-364: Rework symbol representation for speed and size #8343
• BE-360: HashQL: Rework compiletest UI #8340
• BE-302: HashQL: Statement cost analysis for execution target placement #8339
• BE-317: HashQL: Implement TraversalExtraction pass #8331
• BE-315: HashQL: Add GraphReadFilter source variant to MIR #8323
• BE-301: HashQL: Implement size estimation analysis for MIR #8278
• BE-273: HashQL: Profile and benchmark MIR interpreter #8251
• BE-272: HashQL: Add numeric addition and subtraction operators to MIR #8250
• BE-257: HashQL: Implement MIR interpreter with runtime value representation #8245
• BE-271: HashQL: Implement PostInline pass #8240
• BE-269: HashQL: Move PreInline out into Canonicalization pass #8239
• BE-268: HashQL: Rename PreInlining to PreInline for consistency #8238 👈 (View in Graphite)
• BE-270: HashQL: Optimize empty tuples to unit constants in MIR #8237
• BE-197: Implement MIR inline pass for HashQL #8236
• BE-267: Use bump scopes instead of full resets across transformation passes #8234
• BE-266: HashQL: Implement PreInlining pass for MIR optimization #8233
• BE-265: HashQL: Make CopyPropagation aware of local re-assignments #8231
• BE-264: HashQL: Rename SROA to ForwardSubstitution for better semantic clarity #8230
• BE-263: HashQL: add body! macro for declarative MIR construction in tests #8229
• BE-261: HashQL: Implement Administrative Reduction pass for MIR #8227
• BE-262: HashQL: Do not intern empty slices #8223
• BE-260: HashQL: Switch to bump allocator with scoping support #8218
• BE-259: HashQL: Add benchmarks for type system operations #8216
• BE-227: HashQL: Implement call graph analysis for MIR #8214
• BE-258: HashQL: Add Changed enum to track MIR transformation pass modifications #8210
• BE-255: HashQL: Implement Copy Propagation (CP) pass for MIR #8207
• BE-231: HashQL: Add InstSimplify pass for MIR instruction simplification and constant folding #8206
• BE-254: Add CodSpeed benchmarking for HashQL MIR transformation passes #8204
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.73%. Comparing base (4655bbd) to head (447336b).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #8238   +/-   ##
=======================================
  Coverage   59.72%   59.73%           
=======================================
  Files        1214     1214           
  Lines      115245   115275   +30     
  Branches     5062     5063    +1     
=======================================
+ Hits        68832    68861   +29     
- Misses      45611    45612    +1     
  Partials      802      802           

Flag	Coverage Δ
rust.hashql-compiletest	46.65% <ø> (ø)
rust.hashql-mir	89.69% <ø> (+0.02%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[augmentcode]

🤖 Augment PR Summary

Summary: Renames the MIR pre-inlining transform pass to PreInline for naming consistency.

Changes:

• Renamed PreInlining → PreInline and pre_inlining.rs → pre_inline.rs, updating module exports.
• Updated compiletest suites/bench pipelines and helper function names to use the new identifiers.
• Moved UI test fixtures/spec from pre_inlining to pre_inline and updated the suite string to mir/pass/transform/pre-inline.

_{🤖 Was this summary useful? React with 👍 or 👎}

[augmentcode]

Review completed. 1 suggestions posted.

Comment augment review to trigger a new review at any time.

[codspeed-hq]

CodSpeed Performance Report

Merging this PR will not alter performance

_{Comparing bm/be-268-hashql-rename-preinlining-to-preinline (3e63d95) with main (843c87a)}

Summary

✅ 17 untouched benchmarks

[graphite-app]

Merge activity

• Jan 19, 9:35 AM UTC: Graphite rebased this pull request, because this pull request is set to merge when ready.
• Jan 19, 9:35 AM UTC: Graphite rebased this pull request, because this pull request is set to merge when ready.