feat(py, core, llvm): add `is_borrowed` op for BorrowArray by ss2165 · Pull Request #2610 · Quantinuum/hugr

ss2165 · 2025-10-06T11:30:01Z

codecov · 2025-10-06T11:35:01Z

Codecov Report

❌ Patch coverage is 89.13043% with 15 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.35%. Comparing base (4379a0f) to head (af68efd).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
...ugr-llvm/src/extension/collections/borrow_array.rs	87.12%	0 Missing and 13 partials ⚠️
...ore/src/std_extensions/collections/borrow_array.rs	94.59%	0 Missing and 2 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2610      +/-   ##
==========================================
+ Coverage   83.32%   83.35%   +0.02%     
==========================================
  Files         256      256              
  Lines       50491    50618     +127     
  Branches    46014    46141     +127     
==========================================
+ Hits        42074    42195     +121     
+ Misses       6065     6061       -4     
- Partials     2352     2362      +10

Flag	Coverage Δ
python	`91.53% <ø> (ø)`
rust	`82.56% <89.13%> (+0.03%)`	⬆️

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.

acl-cqc

Concerns about use of word "check" is pure pedantry, but I'm not convinced about including this as a MaskCheck

hugr-core/src/std_extensions/collections/borrow_array.rs

acl-cqc · 2025-10-08T15:16:43Z

hugr-core/src/std_extensions/collections/borrow_array.rs

+                DFGBuilder::new(Signature::new(vec![arr_ty.clone()], vec![qb_t(), arr_ty]))
+                    .unwrap();
+            let idx = builder.add_load_value(ConstUsize::new(2));
+            let [arr] = builder.input_wires_arr();


do an is-borrowed before too, and return two booleans

don't see the reason, this is not an execution test, just of building the op

Well I was thinking you could use this in the execution test...

the general pattern is to keep the tests very independent and I would like to maintain that

hugr-llvm/src/extension/collections/borrow_array.rs

hugr-core/src/std_extensions/collections/borrow_array.rs

hugr-llvm/src/extension/collections/borrow_array.rs

hugr-core/src/std_extensions/collections/borrow_array.rs

chore: update extension

aborgna-q · 2025-10-09T12:25:12Z

hugr-core/src/std_extensions/collections/borrow_array.rs

+                    vec![array_ty.clone(), usize_t],
+                    vec![crate::extension::prelude::bool_t(), array_ty],


The type order is a bit weird, with the array in first place for the input but 2nd in the output.
But I see that we're doing the same with borrow so 🤷

## 🤖 New release * `hugr-model`: 0.23.0 -> 0.24.0 * `hugr-core`: 0.23.0 -> 0.24.0 (✓ API compatible changes) * `hugr-llvm`: 0.23.0 -> 0.24.0 (✓ API compatible changes) * `hugr-passes`: 0.23.0 -> 0.24.0 (✓ API compatible changes) * `hugr-persistent`: 0.3.0 -> 0.3.1 (✓ API compatible changes) * `hugr`: 0.23.0 -> 0.24.0 (✓ API compatible changes) * `hugr-cli`: 0.23.0 -> 0.24.0 (✓ API compatible changes) <details><summary>Changelog</summary> ## `hugr-model` <blockquote> ## [0.23.0](hugr-model-v0.22.4...hugr-model-v0.23.0) - 2025-09-30 ### Bug Fixes - [**breaking**] Appease `cargo-audit` by replacing unmaintained dependencies ([#2572](#2572)) ### New Features - Documentation and error hints ([#2523](#2523)) </blockquote> ## `hugr-core` <blockquote> ## [0.24.0](hugr-core-v0.23.0...hugr-core-v0.24.0) - 2025-10-13 ### Bug Fixes - Preserve offset for CFG edges when serializing to JSON ([#2606](#2606)) ### New Features - LLVM lowering for borrow arrays using bitmasks ([#2574](#2574)) - *(py, core, llvm)* add `is_borrowed` op for BorrowArray ([#2610](#2610)) ### Refactor - [**breaking**] consistent inout order in borrow array ([#2621](#2621)) </blockquote> ## `hugr-llvm` <blockquote> ## [0.24.0](hugr-llvm-v0.23.0...hugr-llvm-v0.24.0) - 2025-10-13 ### New Features - LLVM lowering for borrow arrays using bitmasks ([#2574](#2574)) - *(py, core, llvm)* add `is_borrowed` op for BorrowArray ([#2610](#2610)) ### Refactor - [**breaking**] consistent inout order in borrow array ([#2621](#2621)) </blockquote> ## `hugr-passes` <blockquote> ## [0.24.0](hugr-passes-v0.23.0...hugr-passes-v0.24.0) - 2025-10-13 ### New Features - Add handler for copying / discarding borrow arrays to default lineariser ([#2602](#2602)) </blockquote> ## `hugr-persistent` <blockquote> ## [0.3.1](hugr-persistent-v0.3.0...hugr-persistent-v0.3.1) - 2025-10-13 ### Bug Fixes - *(test)* No extension serialisation in persistent-hugr testing ([#2612](#2612)) ### New Features - *(persistent)* Redesign CommitStateSpace, bound Commit lifetime ([#2534](#2534)) </blockquote> ## `hugr` <blockquote> ## [0.24.0](hugr-v0.23.0...hugr-v0.24.0) - 2025-10-13 ### Bug Fixes - Preserve offset for CFG edges when serializing to JSON ([#2606](#2606)) ### New Features - Add handler for copying / discarding borrow arrays to default lineariser ([#2602](#2602)) - LLVM lowering for borrow arrays using bitmasks ([#2574](#2574)) - *(py, core, llvm)* add `is_borrowed` op for BorrowArray ([#2610](#2610)) ### Refactor - [**breaking**] consistent inout order in borrow array ([#2621](#2621)) </blockquote> ## `hugr-cli` <blockquote> ## [0.24.0](hugr-cli-v0.23.0...hugr-cli-v0.24.0) - 2025-10-13 ### Documentation - update cli readme ([#2601](#2601)) </blockquote> </details> --- This PR was generated with [release-plz](https://github.com/release-plz/release-plz/).

🤖 I have created a release *beep* *boop* --- ## [0.14.0](hugr-py-v0.13.1...hugr-py-v0.14.0) (2025-10-13) ### ⚠ BREAKING CHANGES * BorrowArray::{borrow, is_borrowed} return types have been swapped such that the array is first. ### Features * Add more options to `DotRenderer` config ([#2540](#2540)) ([45f7573](45f7573)) * Allow importing from model data to python. ([#2581](#2581)) ([4fb0a5e](4fb0a5e)) * **py, core, llvm:** add `is_borrowed` op for BorrowArray ([#2610](#2610)) ([1cd08ef](1cd08ef)), closes [#2569](#2569) ### Bug Fixes * Preserve offset for CFG edges when serializing to JSON ([#2606](#2606)) ([69a126d](69a126d)) ### Code Refactoring * consistent inout order in borrow array ([#2621](#2621)) ([8fc59f3](8fc59f3)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: Seyon Sivarajah <[email protected]>

## 🤖 New release * `hugr-model`: 0.24.0 -> 0.24.1 (✓ API compatible changes) * `hugr-core`: 0.24.0 -> 0.24.1 (✓ API compatible changes) * `hugr-llvm`: 0.24.0 -> 0.24.1 (✓ API compatible changes) * `hugr-passes`: 0.24.0 -> 0.24.1 (✓ API compatible changes) * `hugr-persistent`: 0.3.1 -> 0.3.2 (✓ API compatible changes) * `hugr`: 0.24.0 -> 0.24.1 (✓ API compatible changes) * `hugr-cli`: 0.24.0 -> 0.24.1 (✓ API compatible changes) <details><summary>Changelog</summary> ## `hugr-model` <blockquote> ## [0.24.1](hugr-model-v0.24.0...hugr-model-v0.24.1) - 2025-11-03 ### Bug Fixes - Correct conversion of `table::Term::Tuple` to `ast::Term` ([#2653](#2653)) </blockquote> ## `hugr-core` <blockquote> ## [0.24.1](hugr-core-v0.24.0...hugr-core-v0.24.1) - 2025-11-03 ### Bug Fixes - validation outside entrypoint, normalize_cfgs w/ nonlocal edges ([#2633](#2633)) - SiblingSubgraph::try_from_nodes for non-entrypoint region ([#2655](#2655)) - Correct conversion of `table::Term::Tuple` to `ast::Term` ([#2653](#2653)) ### New Features - track package descriptions when loading ([#2639](#2639)) - *(cli)* describe sub-command ([#2650](#2650)) </blockquote> ## `hugr-llvm` <blockquote> ## [0.24.0](hugr-llvm-v0.23.0...hugr-llvm-v0.24.0) - 2025-10-13 ### New Features - LLVM lowering for borrow arrays using bitmasks ([#2574](#2574)) - *(py, core, llvm)* add `is_borrowed` op for BorrowArray ([#2610](#2610)) ### Refactor - [**breaking**] consistent inout order in borrow array ([#2621](#2621)) </blockquote> ## `hugr-passes` <blockquote> ## [0.24.1](hugr-passes-v0.24.0...hugr-passes-v0.24.1) - 2025-11-03 ### Bug Fixes - validation outside entrypoint, normalize_cfgs w/ nonlocal edges ([#2633](#2633)) </blockquote> ## `hugr-persistent` <blockquote> ## [0.3.2](hugr-persistent-v0.3.1...hugr-persistent-v0.3.2) - 2025-11-03 ### New Features - *(persistent)* More efficient HugrView iterators for PersistentHugr ([#2595](#2595)) </blockquote> ## `hugr` <blockquote> ## [0.24.1](hugr-v0.24.0...hugr-v0.24.1) - 2025-11-03 ### Bug Fixes - validation outside entrypoint, normalize_cfgs w/ nonlocal edges ([#2633](#2633)) - SiblingSubgraph::try_from_nodes for non-entrypoint region ([#2655](#2655)) - Correct conversion of `table::Term::Tuple` to `ast::Term` ([#2653](#2653)) ### New Features - track package descriptions when loading ([#2639](#2639)) - *(cli)* describe sub-command ([#2650](#2650)) </blockquote> ## `hugr-cli` <blockquote> ## [0.24.1](hugr-cli-v0.24.0...hugr-cli-v0.24.1) - 2025-11-03 ### New Features - track package descriptions when loading ([#2639](#2639)) - *(cli)* describe sub-command ([#2650](#2650)) </blockquote> </details> --- This PR was generated with [release-plz](https://github.com/release-plz/release-plz/).

## 🤖 New release * `hugr-model`: 0.24.1 -> 0.24.2 * `hugr-core`: 0.24.1 -> 0.24.2 * `hugr-llvm`: 0.24.1 -> 0.24.2 * `hugr-passes`: 0.24.1 -> 0.24.2 (✓ API compatible changes) * `hugr`: 0.24.1 -> 0.24.2 (✓ API compatible changes) * `hugr-cli`: 0.24.1 -> 0.24.2 (✓ API compatible changes) * `hugr-persistent`: 0.3.2 -> 0.3.3 <details><summary>Changelog</summary> ## `hugr-model` <blockquote> ## [0.24.1](hugr-model-v0.24.0...hugr-model-v0.24.1) - 2025-11-03 ### Bug Fixes - Correct conversion of `table::Term::Tuple` to `ast::Term` ([#2653](#2653)) </blockquote> ## `hugr-core` <blockquote> ## [0.24.1](hugr-core-v0.24.0...hugr-core-v0.24.1) - 2025-11-03 ### Bug Fixes - validation outside entrypoint, normalize_cfgs w/ nonlocal edges ([#2633](#2633)) - SiblingSubgraph::try_from_nodes for non-entrypoint region ([#2655](#2655)) - Correct conversion of `table::Term::Tuple` to `ast::Term` ([#2653](#2653)) ### New Features - track package descriptions when loading ([#2639](#2639)) - *(cli)* describe sub-command ([#2650](#2650)) </blockquote> ## `hugr-llvm` <blockquote> ## [0.24.0](hugr-llvm-v0.23.0...hugr-llvm-v0.24.0) - 2025-10-13 ### New Features - LLVM lowering for borrow arrays using bitmasks ([#2574](#2574)) - *(py, core, llvm)* add `is_borrowed` op for BorrowArray ([#2610](#2610)) ### Refactor - [**breaking**] consistent inout order in borrow array ([#2621](#2621)) </blockquote> ## `hugr-passes` <blockquote> ## [0.24.2](hugr-passes-v0.24.1...hugr-passes-v0.24.2) - 2025-11-03 ### Bug Fixes - ReplaceTypes: operate on whole Hugr, with set_regions ([#2662](#2662)) </blockquote> ## `hugr` <blockquote> ## [0.24.2](hugr-v0.24.1...hugr-v0.24.2) - 2025-11-03 ### Bug Fixes - ReplaceTypes: operate on whole Hugr, with set_regions ([#2662](#2662)) </blockquote> ## `hugr-cli` <blockquote> ## [0.24.1](hugr-cli-v0.24.0...hugr-cli-v0.24.1) - 2025-11-03 ### New Features - track package descriptions when loading ([#2639](#2639)) - *(cli)* describe sub-command ([#2650](#2650)) </blockquote> ## `hugr-persistent` <blockquote> ## [0.3.2](hugr-persistent-v0.3.1...hugr-persistent-v0.3.2) - 2025-11-03 ### New Features - *(persistent)* More efficient HugrView iterators for PersistentHugr ([#2595](#2595)) </blockquote> </details> --- This PR was generated with [release-plz](https://github.com/release-plz/release-plz/).

ss2165 requested a review from a team as a code owner October 6, 2025 11:30

ss2165 requested review from lmondada and mark-koch and removed request for a team and lmondada October 6, 2025 11:30

Base automatically changed from ss/push-owxrxomqozmv to feat/barray-llvm October 6, 2025 12:55

ss2165 force-pushed the ss/push-vnnxrlmwsvky branch from 3541254 to eedf03d Compare October 6, 2025 12:56

ss2165 changed the title ~~feat(core, llvm): add is_borrowed op for BorrowArray~~ feat(py, core, llvm): add is_borrowed op for BorrowArray Oct 6, 2025

acl-cqc force-pushed the feat/barray-llvm branch from 91f42b2 to 439f09c Compare October 6, 2025 14:22

ss2165 force-pushed the ss/push-vnnxrlmwsvky branch from 6556085 to 82ea2fe Compare October 6, 2025 15:07

acl-cqc force-pushed the feat/barray-llvm branch 2 times, most recently from 5be9249 to d5d5999 Compare October 7, 2025 14:59

Base automatically changed from feat/barray-llvm to main October 8, 2025 13:08

ss2165 force-pushed the ss/push-vnnxrlmwsvky branch from 82ea2fe to ed70a19 Compare October 8, 2025 15:09

ss2165 requested review from acl-cqc and removed request for mark-koch October 8, 2025 15:09

acl-cqc reviewed Oct 8, 2025

View reviewed changes

ss2165 force-pushed the ss/push-vnnxrlmwsvky branch from ed70a19 to bb9384d Compare October 8, 2025 17:15

ss2165 requested review from aborgna-q and acl-cqc October 8, 2025 17:16

ss2165 added 2 commits October 9, 2025 10:53

feat(core): add borrow_array is_borrowed op

4bcec25

chore: update extension

feat: add llvm codegen for is_borrowed

af68efd

ss2165 force-pushed the ss/push-vnnxrlmwsvky branch from bb9384d to af68efd Compare October 9, 2025 09:54

aborgna-q approved these changes Oct 9, 2025

View reviewed changes

ss2165 added this pull request to the merge queue Oct 10, 2025

Merged via the queue into main with commit 1cd08ef Oct 10, 2025
26 checks passed

ss2165 deleted the ss/push-vnnxrlmwsvky branch October 10, 2025 15:25

hugrbot mentioned this pull request Oct 8, 2025

chore(main): release hugr-py 0.14.0 #2547

Merged

hugrbot mentioned this pull request Oct 8, 2025

chore: release #2605

Merged

hugrbot mentioned this pull request Oct 14, 2025

chore: release #2625

Merged

hugrbot mentioned this pull request Nov 3, 2025

chore: release #2661

Merged

hugrbot mentioned this pull request Mar 16, 2026

chore(main): release hugr-py 0.16.0 #2948

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(py, core, llvm): add `is_borrowed` op for BorrowArray#2610

feat(py, core, llvm): add `is_borrowed` op for BorrowArray#2610
ss2165 merged 2 commits intomainfrom
ss/push-vnnxrlmwsvky

ss2165 commented Oct 6, 2025 •

edited

Loading

Uh oh!

codecov bot commented Oct 6, 2025 •

edited

Loading

Uh oh!

acl-cqc left a comment

Uh oh!

Uh oh!

acl-cqc Oct 8, 2025

Uh oh!

ss2165 Oct 8, 2025

Uh oh!

acl-cqc Oct 8, 2025

Uh oh!

ss2165 Oct 8, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

aborgna-q Oct 9, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

		vec![array_ty.clone(), usize_t],
		vec![crate::extension::prelude::bool_t(), array_ty],

Conversation

ss2165 commented Oct 6, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

codecov bot commented Oct 6, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

acl-cqc left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

acl-cqc Oct 8, 2025

Choose a reason for hiding this comment

Uh oh!

ss2165 Oct 8, 2025

Choose a reason for hiding this comment

Uh oh!

acl-cqc Oct 8, 2025

Choose a reason for hiding this comment

Uh oh!

ss2165 Oct 8, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

aborgna-q Oct 9, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

ss2165 commented Oct 6, 2025 •

edited

Loading

codecov bot commented Oct 6, 2025 •

edited

Loading