Emit thumb code on VEX V5 - #158762

Merged
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
vexide:feat/vex-v5-thumb2
Aug 5, 2026
Merged

Emit thumb code on VEX V5#158762
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
vexide:feat/vex-v5-thumb2

Conversation

@lewisfm

@lewisfmlewisfm commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

This PR switches the default codegen for the VEX V5 target to emit Thumb-2 instructions, allowing for smaller binary sizes (on the programs I tested this change on, I saw a ~20% size decrease on average). The target is renamed to have the thumb prefix instead of the arm prefix because of the updated instruction set.

Since VEXos starts all programs in Arm32 mode, the program entrypoint is explicitly compiled as ARM code and now transitions to Thumb mode when calling _start.

Users can still use the updated target in Arm32 mode by specifying -Ctarget-feature=-thumb-mode.

cc @tropicaaal@Gavin-Niederman

@rustbot

Copy link
Copy Markdown
Collaborator

Some changes occurred in src/doc/rustc/src/platform-support

cc @Noratrieb

These commits modify compiler targets.
(See the Target Tier Policy.)

@rustbotrustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Jul 4, 2026
@rustbot

Copy link
Copy Markdown
Collaborator

r? @Kivooeo

rustbot has assigned @Kivooeo.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: @ehuss, compiler
  • @ehuss, compiler expanded to 76 candidates
  • Random selection from 23 candidates

@rust-log-analyzer

Copy link
Copy Markdown
Collaborator

A job failed! Check out the build log: (web)(plain enhanced)(plain)

Click to see the possible cause of the failure (guessed by this bot)
 PR_CI_JOB: 1
IMAGE: x86_64-gnu-llvm-21
##[endgroup]
Updating crates.io index
error: failed to get `fnv` as a dependency of package `http v1.2.0`
... which satisfies dependency `http = "^1.1.0"` of package `ureq-proto v0.3.3`
... which satisfies dependency `ureq-proto = "^0.3.3"` of package `ureq v3.0.8`
... which satisfies dependency `ureq = "^3"` of package `citool v0.1.0 (/home/runner/work/rust/rust/src/ci/citool)`
Caused by:
failed to load source for dependency `fnv`
Caused by:
unable to update registry `crates-io`
Caused by:
download of 3/f/fnv failed
Caused by:
curl failed
Caused by:

@Gavin-NiedermanGavin-Niederman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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


**Tier: 3**

Allows compiling user programs for the [VEX V5 Brain](https://www.vexrobotics.com/276-4810.html), a microcontroller for educational and competitive robotics.

Rust support for this target is not affiliated with VEX Robotics or IFI, and does not link against any official VEX SDK.

This target was previously named `armv7a-vex-v5`.

@tropicaaaltropicaaalJul 11, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Not a blocker, but it might be a good idea for this to be elaborated on wrt. why we changed to thumb.

View changes since the review

@tropicaaaltropicaaal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@Kivooeo

Kivooeo commented Aug 5, 2026

Copy link
Copy Markdown
Member

@rustbot reroll (don't know anything about this targets and don't have bandwidth to figure it out)

@rustbotrustbot assigned jieyouxu and unassigned KivooeoAug 5, 2026
@jieyouxujieyouxu closed this Aug 5, 2026
@rustbotrustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 5, 2026
@jieyouxujieyouxu reopened this Aug 5, 2026
@rustbotrustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 5, 2026
@rustbot

Copy link
Copy Markdown
Collaborator

⚠️Warning⚠️

@jieyouxujieyouxu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks. Given that this is a Tier 3 target and target maintainers are on board I'm happy to stamp this.

@bors r+ rollup

View changes since this review

@rust-bors

rust-borsBot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 383d557 has been approved by jieyouxu

It is now in the queue for this repository.

@rust-borsrust-borsBot added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 5, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Aug 5, 2026
Emit thumb code on VEX V5
This PR switches the default codegen for the VEX V5 target to emit Thumb-2 instructions, allowing for smaller binary sizes (on the programs I tested this change on, I saw a ~20% size decrease on average). The target is renamed to have the `thumb` prefix instead of the `arm` prefix because of the updated instruction set.
Since VEXos starts all programs in Arm32 mode, the program entrypoint is explicitly compiled as ARM code and now transitions to Thumb mode when calling `_start`.
Users can still use the updated target in Arm32 mode by specifying `-Ctarget-feature=-thumb-mode`.
cc @tropicaaal@Gavin-Niederman
rust-borsBot pushed a commit that referenced this pull request Aug 5, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- #154585 (treat no_mangle_generic_items as hard error instead of lint warning)
- #158147 (std: fix stack buffer overflow in Windows junction_point)
- #160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- #160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- #160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- #160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- #160422 (move mir-opt miri tests to CI logic)
- #160444 (Avoid resolving path keywords outside `TypeNS`)
- #160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe my unsafe speculative flag)
- #155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- #158726 (std: move futex implementations into sys::sync::futex)
- #158762 (Emit thumb code on VEX V5)
- #159225 (Split IncrCompSession out of Session)
- #159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- #160198 (Rework `smallest_range_containing` to handle duplicates)
- #160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- #160390 (autodiff: Handle slice-tailed DSTs in type trees)
- #160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- #160516 (Add regression test for HRTB projection in closure)
- #160520 (Add some tests for specialization)
- #160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- #160523 (Add regression test for opaque type)
- #160531 (docs: fix typo in AllowExprMetavar comment)
- #160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- #160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
rust-borsBot pushed a commit that referenced this pull request Aug 5, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- #158147 (std: fix stack buffer overflow in Windows junction_point)
- #160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- #160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- #160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- #160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- #160422 (move mir-opt miri tests to CI logic)
- #160444 (Avoid resolving path keywords outside `TypeNS`)
- #160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe by unsafe speculative flag)
- #155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- #158726 (std: move futex implementations into sys::sync::futex)
- #158762 (Emit thumb code on VEX V5)
- #159225 (Split IncrCompSession out of Session)
- #159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- #160198 (Rework `smallest_range_containing` to handle duplicates)
- #160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- #160390 (autodiff: Handle slice-tailed DSTs in type trees)
- #160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- #160501 (Add bootstrap CLI snapshot test for testing miri)
- #160516 (Add regression test for HRTB projection in closure)
- #160520 (Add some tests for specialization)
- #160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- #160523 (Add regression test for opaque type)
- #160531 (docs: fix typo in AllowExprMetavar comment)
- #160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- #160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
@rust-bors
rust-borsBot merged commit 8c6b914 into rust-lang:mainAug 5, 2026
15 of 26 checks passed
@rustbotrustbot added this to the 1.99.0 milestone Aug 5, 2026
rust-timer added a commit that referenced this pull request Aug 5, 2026
Rollup merge of #158762 - vexide:feat/vex-v5-thumb2, r=jieyouxu
Emit thumb code on VEX V5
This PR switches the default codegen for the VEX V5 target to emit Thumb-2 instructions, allowing for smaller binary sizes (on the programs I tested this change on, I saw a ~20% size decrease on average). The target is renamed to have the `thumb` prefix instead of the `arm` prefix because of the updated instruction set.
Since VEXos starts all programs in Arm32 mode, the program entrypoint is explicitly compiled as ARM code and now transitions to Thumb mode when calling `_start`.
Users can still use the updated target in Arm32 mode by specifying `-Ctarget-feature=-thumb-mode`.
cc @tropicaaal@Gavin-Niederman
github-actionsBot pushed a commit to rust-lang/rustc-dev-guide that referenced this pull request Aug 10, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- rust-lang/rust#158147 (std: fix stack buffer overflow in Windows junction_point)
- rust-lang/rust#160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- rust-lang/rust#160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- rust-lang/rust#160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- rust-lang/rust#160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- rust-lang/rust#160422 (move mir-opt miri tests to CI logic)
- rust-lang/rust#160444 (Avoid resolving path keywords outside `TypeNS`)
- rust-lang/rust#160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe by unsafe speculative flag)
- rust-lang/rust#155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- rust-lang/rust#158726 (std: move futex implementations into sys::sync::futex)
- rust-lang/rust#158762 (Emit thumb code on VEX V5)
- rust-lang/rust#159225 (Split IncrCompSession out of Session)
- rust-lang/rust#159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- rust-lang/rust#160198 (Rework `smallest_range_containing` to handle duplicates)
- rust-lang/rust#160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- rust-lang/rust#160390 (autodiff: Handle slice-tailed DSTs in type trees)
- rust-lang/rust#160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- rust-lang/rust#160501 (Add bootstrap CLI snapshot test for testing miri)
- rust-lang/rust#160516 (Add regression test for HRTB projection in closure)
- rust-lang/rust#160520 (Add some tests for specialization)
- rust-lang/rust#160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- rust-lang/rust#160523 (Add regression test for opaque type)
- rust-lang/rust#160531 (docs: fix typo in AllowExprMetavar comment)
- rust-lang/rust#160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- rust-lang/rust#160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
flip1995 pushed a commit to flip1995/rust-clippy that referenced this pull request Aug 17, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- rust-lang/rust#158147 (std: fix stack buffer overflow in Windows junction_point)
- rust-lang/rust#160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- rust-lang/rust#160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- rust-lang/rust#160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- rust-lang/rust#160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- rust-lang/rust#160422 (move mir-opt miri tests to CI logic)
- rust-lang/rust#160444 (Avoid resolving path keywords outside `TypeNS`)
- rust-lang/rust#160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe by unsafe speculative flag)
- rust-lang/rust#155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- rust-lang/rust#158726 (std: move futex implementations into sys::sync::futex)
- rust-lang/rust#158762 (Emit thumb code on VEX V5)
- rust-lang/rust#159225 (Split IncrCompSession out of Session)
- rust-lang/rust#159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- rust-lang/rust#160198 (Rework `smallest_range_containing` to handle duplicates)
- rust-lang/rust#160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- rust-lang/rust#160390 (autodiff: Handle slice-tailed DSTs in type trees)
- rust-lang/rust#160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- rust-lang/rust#160501 (Add bootstrap CLI snapshot test for testing miri)
- rust-lang/rust#160516 (Add regression test for HRTB projection in closure)
- rust-lang/rust#160520 (Add some tests for specialization)
- rust-lang/rust#160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- rust-lang/rust#160523 (Add regression test for opaque type)
- rust-lang/rust#160531 (docs: fix typo in AllowExprMetavar comment)
- rust-lang/rust#160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- rust-lang/rust#160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.T-libsRelevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@lewisfm@rustbot@rust-log-analyzer@Kivooeo@jieyouxu@tropicaaal@Gavin-Niederman
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
Skip to content

Emit thumb code on VEX V5 - #158762

Merged
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
vexide:feat/vex-v5-thumb2
Aug 5, 2026
Merged

Emit thumb code on VEX V5#158762
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
vexide:feat/vex-v5-thumb2

Conversation

@lewisfm

@lewisfmlewisfm commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

This PR switches the default codegen for the VEX V5 target to emit Thumb-2 instructions, allowing for smaller binary sizes (on the programs I tested this change on, I saw a ~20% size decrease on average). The target is renamed to have the thumb prefix instead of the arm prefix because of the updated instruction set.

Since VEXos starts all programs in Arm32 mode, the program entrypoint is explicitly compiled as ARM code and now transitions to Thumb mode when calling _start.

Users can still use the updated target in Arm32 mode by specifying -Ctarget-feature=-thumb-mode.

cc @tropicaaal@Gavin-Niederman

@rustbot

Copy link
Copy Markdown
Collaborator

Some changes occurred in src/doc/rustc/src/platform-support

cc @Noratrieb

These commits modify compiler targets.
(See the Target Tier Policy.)

@rustbotrustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Jul 4, 2026
@rustbot

Copy link
Copy Markdown
Collaborator

r? @Kivooeo

rustbot has assigned @Kivooeo.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: @ehuss, compiler
  • @ehuss, compiler expanded to 76 candidates
  • Random selection from 23 candidates

@rust-log-analyzer

Copy link
Copy Markdown
Collaborator

A job failed! Check out the build log: (web)(plain enhanced)(plain)

Click to see the possible cause of the failure (guessed by this bot)
 PR_CI_JOB: 1
IMAGE: x86_64-gnu-llvm-21
##[endgroup]
Updating crates.io index
error: failed to get `fnv` as a dependency of package `http v1.2.0`
... which satisfies dependency `http = "^1.1.0"` of package `ureq-proto v0.3.3`
... which satisfies dependency `ureq-proto = "^0.3.3"` of package `ureq v3.0.8`
... which satisfies dependency `ureq = "^3"` of package `citool v0.1.0 (/home/runner/work/rust/rust/src/ci/citool)`
Caused by:
failed to load source for dependency `fnv`
Caused by:
unable to update registry `crates-io`
Caused by:
download of 3/f/fnv failed
Caused by:
curl failed
Caused by:

@Gavin-NiedermanGavin-Niederman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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


**Tier: 3**

Allows compiling user programs for the [VEX V5 Brain](https://www.vexrobotics.com/276-4810.html), a microcontroller for educational and competitive robotics.

Rust support for this target is not affiliated with VEX Robotics or IFI, and does not link against any official VEX SDK.

This target was previously named `armv7a-vex-v5`.

@tropicaaaltropicaaalJul 11, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Not a blocker, but it might be a good idea for this to be elaborated on wrt. why we changed to thumb.

View changes since the review

@tropicaaaltropicaaal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@Kivooeo

Kivooeo commented Aug 5, 2026

Copy link
Copy Markdown
Member

@rustbot reroll (don't know anything about this targets and don't have bandwidth to figure it out)

@rustbotrustbot assigned jieyouxu and unassigned KivooeoAug 5, 2026
@jieyouxujieyouxu closed this Aug 5, 2026
@rustbotrustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 5, 2026
@jieyouxujieyouxu reopened this Aug 5, 2026
@rustbotrustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 5, 2026
@rustbot

Copy link
Copy Markdown
Collaborator

⚠️Warning⚠️

@jieyouxujieyouxu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks. Given that this is a Tier 3 target and target maintainers are on board I'm happy to stamp this.

@bors r+ rollup

View changes since this review

@rust-bors

rust-borsBot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 383d557 has been approved by jieyouxu

It is now in the queue for this repository.

@rust-borsrust-borsBot added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 5, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Aug 5, 2026
Emit thumb code on VEX V5
This PR switches the default codegen for the VEX V5 target to emit Thumb-2 instructions, allowing for smaller binary sizes (on the programs I tested this change on, I saw a ~20% size decrease on average). The target is renamed to have the `thumb` prefix instead of the `arm` prefix because of the updated instruction set.
Since VEXos starts all programs in Arm32 mode, the program entrypoint is explicitly compiled as ARM code and now transitions to Thumb mode when calling `_start`.
Users can still use the updated target in Arm32 mode by specifying `-Ctarget-feature=-thumb-mode`.
cc @tropicaaal@Gavin-Niederman
rust-borsBot pushed a commit that referenced this pull request Aug 5, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- #154585 (treat no_mangle_generic_items as hard error instead of lint warning)
- #158147 (std: fix stack buffer overflow in Windows junction_point)
- #160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- #160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- #160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- #160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- #160422 (move mir-opt miri tests to CI logic)
- #160444 (Avoid resolving path keywords outside `TypeNS`)
- #160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe my unsafe speculative flag)
- #155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- #158726 (std: move futex implementations into sys::sync::futex)
- #158762 (Emit thumb code on VEX V5)
- #159225 (Split IncrCompSession out of Session)
- #159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- #160198 (Rework `smallest_range_containing` to handle duplicates)
- #160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- #160390 (autodiff: Handle slice-tailed DSTs in type trees)
- #160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- #160516 (Add regression test for HRTB projection in closure)
- #160520 (Add some tests for specialization)
- #160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- #160523 (Add regression test for opaque type)
- #160531 (docs: fix typo in AllowExprMetavar comment)
- #160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- #160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
rust-borsBot pushed a commit that referenced this pull request Aug 5, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- #158147 (std: fix stack buffer overflow in Windows junction_point)
- #160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- #160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- #160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- #160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- #160422 (move mir-opt miri tests to CI logic)
- #160444 (Avoid resolving path keywords outside `TypeNS`)
- #160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe by unsafe speculative flag)
- #155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- #158726 (std: move futex implementations into sys::sync::futex)
- #158762 (Emit thumb code on VEX V5)
- #159225 (Split IncrCompSession out of Session)
- #159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- #160198 (Rework `smallest_range_containing` to handle duplicates)
- #160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- #160390 (autodiff: Handle slice-tailed DSTs in type trees)
- #160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- #160501 (Add bootstrap CLI snapshot test for testing miri)
- #160516 (Add regression test for HRTB projection in closure)
- #160520 (Add some tests for specialization)
- #160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- #160523 (Add regression test for opaque type)
- #160531 (docs: fix typo in AllowExprMetavar comment)
- #160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- #160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
@rust-bors
rust-borsBot merged commit 8c6b914 into rust-lang:mainAug 5, 2026
15 of 26 checks passed
@rustbotrustbot added this to the 1.99.0 milestone Aug 5, 2026
rust-timer added a commit that referenced this pull request Aug 5, 2026
Rollup merge of #158762 - vexide:feat/vex-v5-thumb2, r=jieyouxu
Emit thumb code on VEX V5
This PR switches the default codegen for the VEX V5 target to emit Thumb-2 instructions, allowing for smaller binary sizes (on the programs I tested this change on, I saw a ~20% size decrease on average). The target is renamed to have the `thumb` prefix instead of the `arm` prefix because of the updated instruction set.
Since VEXos starts all programs in Arm32 mode, the program entrypoint is explicitly compiled as ARM code and now transitions to Thumb mode when calling `_start`.
Users can still use the updated target in Arm32 mode by specifying `-Ctarget-feature=-thumb-mode`.
cc @tropicaaal@Gavin-Niederman
github-actionsBot pushed a commit to rust-lang/rustc-dev-guide that referenced this pull request Aug 10, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- rust-lang/rust#158147 (std: fix stack buffer overflow in Windows junction_point)
- rust-lang/rust#160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- rust-lang/rust#160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- rust-lang/rust#160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- rust-lang/rust#160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- rust-lang/rust#160422 (move mir-opt miri tests to CI logic)
- rust-lang/rust#160444 (Avoid resolving path keywords outside `TypeNS`)
- rust-lang/rust#160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe by unsafe speculative flag)
- rust-lang/rust#155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- rust-lang/rust#158726 (std: move futex implementations into sys::sync::futex)
- rust-lang/rust#158762 (Emit thumb code on VEX V5)
- rust-lang/rust#159225 (Split IncrCompSession out of Session)
- rust-lang/rust#159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- rust-lang/rust#160198 (Rework `smallest_range_containing` to handle duplicates)
- rust-lang/rust#160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- rust-lang/rust#160390 (autodiff: Handle slice-tailed DSTs in type trees)
- rust-lang/rust#160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- rust-lang/rust#160501 (Add bootstrap CLI snapshot test for testing miri)
- rust-lang/rust#160516 (Add regression test for HRTB projection in closure)
- rust-lang/rust#160520 (Add some tests for specialization)
- rust-lang/rust#160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- rust-lang/rust#160523 (Add regression test for opaque type)
- rust-lang/rust#160531 (docs: fix typo in AllowExprMetavar comment)
- rust-lang/rust#160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- rust-lang/rust#160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
flip1995 pushed a commit to flip1995/rust-clippy that referenced this pull request Aug 17, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- rust-lang/rust#158147 (std: fix stack buffer overflow in Windows junction_point)
- rust-lang/rust#160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- rust-lang/rust#160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- rust-lang/rust#160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- rust-lang/rust#160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- rust-lang/rust#160422 (move mir-opt miri tests to CI logic)
- rust-lang/rust#160444 (Avoid resolving path keywords outside `TypeNS`)
- rust-lang/rust#160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe by unsafe speculative flag)
- rust-lang/rust#155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- rust-lang/rust#158726 (std: move futex implementations into sys::sync::futex)
- rust-lang/rust#158762 (Emit thumb code on VEX V5)
- rust-lang/rust#159225 (Split IncrCompSession out of Session)
- rust-lang/rust#159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- rust-lang/rust#160198 (Rework `smallest_range_containing` to handle duplicates)
- rust-lang/rust#160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- rust-lang/rust#160390 (autodiff: Handle slice-tailed DSTs in type trees)
- rust-lang/rust#160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- rust-lang/rust#160501 (Add bootstrap CLI snapshot test for testing miri)
- rust-lang/rust#160516 (Add regression test for HRTB projection in closure)
- rust-lang/rust#160520 (Add some tests for specialization)
- rust-lang/rust#160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- rust-lang/rust#160523 (Add regression test for opaque type)
- rust-lang/rust#160531 (docs: fix typo in AllowExprMetavar comment)
- rust-lang/rust#160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- rust-lang/rust#160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.T-libsRelevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@lewisfm@rustbot@rust-log-analyzer@Kivooeo@jieyouxu@tropicaaal@Gavin-Niederman
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix"); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

Emit thumb code on VEX V5 - #158762

Merged
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
vexide:feat/vex-v5-thumb2
Aug 5, 2026
Merged

Emit thumb code on VEX V5#158762
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
vexide:feat/vex-v5-thumb2

Conversation

@lewisfm

@lewisfmlewisfm commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

This PR switches the default codegen for the VEX V5 target to emit Thumb-2 instructions, allowing for smaller binary sizes (on the programs I tested this change on, I saw a ~20% size decrease on average). The target is renamed to have the thumb prefix instead of the arm prefix because of the updated instruction set.

Since VEXos starts all programs in Arm32 mode, the program entrypoint is explicitly compiled as ARM code and now transitions to Thumb mode when calling _start.

Users can still use the updated target in Arm32 mode by specifying -Ctarget-feature=-thumb-mode.

cc @tropicaaal@Gavin-Niederman

@rustbot

Copy link
Copy Markdown
Collaborator

Some changes occurred in src/doc/rustc/src/platform-support

cc @Noratrieb

These commits modify compiler targets.
(See the Target Tier Policy.)

@rustbotrustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Jul 4, 2026
@rustbot

Copy link
Copy Markdown
Collaborator

r? @Kivooeo

rustbot has assigned @Kivooeo.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: @ehuss, compiler
  • @ehuss, compiler expanded to 76 candidates
  • Random selection from 23 candidates

@rust-log-analyzer

Copy link
Copy Markdown
Collaborator

A job failed! Check out the build log: (web)(plain enhanced)(plain)

Click to see the possible cause of the failure (guessed by this bot)
 PR_CI_JOB: 1
IMAGE: x86_64-gnu-llvm-21
##[endgroup]
Updating crates.io index
error: failed to get `fnv` as a dependency of package `http v1.2.0`
... which satisfies dependency `http = "^1.1.0"` of package `ureq-proto v0.3.3`
... which satisfies dependency `ureq-proto = "^0.3.3"` of package `ureq v3.0.8`
... which satisfies dependency `ureq = "^3"` of package `citool v0.1.0 (/home/runner/work/rust/rust/src/ci/citool)`
Caused by:
failed to load source for dependency `fnv`
Caused by:
unable to update registry `crates-io`
Caused by:
download of 3/f/fnv failed
Caused by:
curl failed
Caused by:

@Gavin-NiedermanGavin-Niederman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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


**Tier: 3**

Allows compiling user programs for the [VEX V5 Brain](https://www.vexrobotics.com/276-4810.html), a microcontroller for educational and competitive robotics.

Rust support for this target is not affiliated with VEX Robotics or IFI, and does not link against any official VEX SDK.

This target was previously named `armv7a-vex-v5`.

@tropicaaaltropicaaalJul 11, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Not a blocker, but it might be a good idea for this to be elaborated on wrt. why we changed to thumb.

View changes since the review

@tropicaaaltropicaaal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@Kivooeo

Kivooeo commented Aug 5, 2026

Copy link
Copy Markdown
Member

@rustbot reroll (don't know anything about this targets and don't have bandwidth to figure it out)

@rustbotrustbot assigned jieyouxu and unassigned KivooeoAug 5, 2026
@jieyouxujieyouxu closed this Aug 5, 2026
@rustbotrustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 5, 2026
@jieyouxujieyouxu reopened this Aug 5, 2026
@rustbotrustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 5, 2026
@rustbot

Copy link
Copy Markdown
Collaborator

⚠️Warning⚠️

@jieyouxujieyouxu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks. Given that this is a Tier 3 target and target maintainers are on board I'm happy to stamp this.

@bors r+ rollup

View changes since this review

@rust-bors

rust-borsBot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 383d557 has been approved by jieyouxu

It is now in the queue for this repository.

@rust-borsrust-borsBot added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 5, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Aug 5, 2026
Emit thumb code on VEX V5
This PR switches the default codegen for the VEX V5 target to emit Thumb-2 instructions, allowing for smaller binary sizes (on the programs I tested this change on, I saw a ~20% size decrease on average). The target is renamed to have the `thumb` prefix instead of the `arm` prefix because of the updated instruction set.
Since VEXos starts all programs in Arm32 mode, the program entrypoint is explicitly compiled as ARM code and now transitions to Thumb mode when calling `_start`.
Users can still use the updated target in Arm32 mode by specifying `-Ctarget-feature=-thumb-mode`.
cc @tropicaaal@Gavin-Niederman
rust-borsBot pushed a commit that referenced this pull request Aug 5, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- #154585 (treat no_mangle_generic_items as hard error instead of lint warning)
- #158147 (std: fix stack buffer overflow in Windows junction_point)
- #160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- #160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- #160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- #160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- #160422 (move mir-opt miri tests to CI logic)
- #160444 (Avoid resolving path keywords outside `TypeNS`)
- #160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe my unsafe speculative flag)
- #155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- #158726 (std: move futex implementations into sys::sync::futex)
- #158762 (Emit thumb code on VEX V5)
- #159225 (Split IncrCompSession out of Session)
- #159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- #160198 (Rework `smallest_range_containing` to handle duplicates)
- #160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- #160390 (autodiff: Handle slice-tailed DSTs in type trees)
- #160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- #160516 (Add regression test for HRTB projection in closure)
- #160520 (Add some tests for specialization)
- #160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- #160523 (Add regression test for opaque type)
- #160531 (docs: fix typo in AllowExprMetavar comment)
- #160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- #160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
rust-borsBot pushed a commit that referenced this pull request Aug 5, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- #158147 (std: fix stack buffer overflow in Windows junction_point)
- #160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- #160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- #160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- #160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- #160422 (move mir-opt miri tests to CI logic)
- #160444 (Avoid resolving path keywords outside `TypeNS`)
- #160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe by unsafe speculative flag)
- #155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- #158726 (std: move futex implementations into sys::sync::futex)
- #158762 (Emit thumb code on VEX V5)
- #159225 (Split IncrCompSession out of Session)
- #159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- #160198 (Rework `smallest_range_containing` to handle duplicates)
- #160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- #160390 (autodiff: Handle slice-tailed DSTs in type trees)
- #160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- #160501 (Add bootstrap CLI snapshot test for testing miri)
- #160516 (Add regression test for HRTB projection in closure)
- #160520 (Add some tests for specialization)
- #160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- #160523 (Add regression test for opaque type)
- #160531 (docs: fix typo in AllowExprMetavar comment)
- #160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- #160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
@rust-bors
rust-borsBot merged commit 8c6b914 into rust-lang:mainAug 5, 2026
15 of 26 checks passed
@rustbotrustbot added this to the 1.99.0 milestone Aug 5, 2026
rust-timer added a commit that referenced this pull request Aug 5, 2026
Rollup merge of #158762 - vexide:feat/vex-v5-thumb2, r=jieyouxu
Emit thumb code on VEX V5
This PR switches the default codegen for the VEX V5 target to emit Thumb-2 instructions, allowing for smaller binary sizes (on the programs I tested this change on, I saw a ~20% size decrease on average). The target is renamed to have the `thumb` prefix instead of the `arm` prefix because of the updated instruction set.
Since VEXos starts all programs in Arm32 mode, the program entrypoint is explicitly compiled as ARM code and now transitions to Thumb mode when calling `_start`.
Users can still use the updated target in Arm32 mode by specifying `-Ctarget-feature=-thumb-mode`.
cc @tropicaaal@Gavin-Niederman
github-actionsBot pushed a commit to rust-lang/rustc-dev-guide that referenced this pull request Aug 10, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- rust-lang/rust#158147 (std: fix stack buffer overflow in Windows junction_point)
- rust-lang/rust#160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- rust-lang/rust#160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- rust-lang/rust#160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- rust-lang/rust#160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- rust-lang/rust#160422 (move mir-opt miri tests to CI logic)
- rust-lang/rust#160444 (Avoid resolving path keywords outside `TypeNS`)
- rust-lang/rust#160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe by unsafe speculative flag)
- rust-lang/rust#155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- rust-lang/rust#158726 (std: move futex implementations into sys::sync::futex)
- rust-lang/rust#158762 (Emit thumb code on VEX V5)
- rust-lang/rust#159225 (Split IncrCompSession out of Session)
- rust-lang/rust#159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- rust-lang/rust#160198 (Rework `smallest_range_containing` to handle duplicates)
- rust-lang/rust#160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- rust-lang/rust#160390 (autodiff: Handle slice-tailed DSTs in type trees)
- rust-lang/rust#160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- rust-lang/rust#160501 (Add bootstrap CLI snapshot test for testing miri)
- rust-lang/rust#160516 (Add regression test for HRTB projection in closure)
- rust-lang/rust#160520 (Add some tests for specialization)
- rust-lang/rust#160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- rust-lang/rust#160523 (Add regression test for opaque type)
- rust-lang/rust#160531 (docs: fix typo in AllowExprMetavar comment)
- rust-lang/rust#160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- rust-lang/rust#160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
flip1995 pushed a commit to flip1995/rust-clippy that referenced this pull request Aug 17, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- rust-lang/rust#158147 (std: fix stack buffer overflow in Windows junction_point)
- rust-lang/rust#160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- rust-lang/rust#160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- rust-lang/rust#160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- rust-lang/rust#160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- rust-lang/rust#160422 (move mir-opt miri tests to CI logic)
- rust-lang/rust#160444 (Avoid resolving path keywords outside `TypeNS`)
- rust-lang/rust#160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe by unsafe speculative flag)
- rust-lang/rust#155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- rust-lang/rust#158726 (std: move futex implementations into sys::sync::futex)
- rust-lang/rust#158762 (Emit thumb code on VEX V5)
- rust-lang/rust#159225 (Split IncrCompSession out of Session)
- rust-lang/rust#159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- rust-lang/rust#160198 (Rework `smallest_range_containing` to handle duplicates)
- rust-lang/rust#160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- rust-lang/rust#160390 (autodiff: Handle slice-tailed DSTs in type trees)
- rust-lang/rust#160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- rust-lang/rust#160501 (Add bootstrap CLI snapshot test for testing miri)
- rust-lang/rust#160516 (Add regression test for HRTB projection in closure)
- rust-lang/rust#160520 (Add some tests for specialization)
- rust-lang/rust#160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- rust-lang/rust#160523 (Add regression test for opaque type)
- rust-lang/rust#160531 (docs: fix typo in AllowExprMetavar comment)
- rust-lang/rust#160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- rust-lang/rust#160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.T-libsRelevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@lewisfm@rustbot@rust-log-analyzer@Kivooeo@jieyouxu@tropicaaal@Gavin-Niederman
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms"); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

Emit thumb code on VEX V5 - #158762

Merged
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
vexide:feat/vex-v5-thumb2
Aug 5, 2026
Merged

Emit thumb code on VEX V5#158762
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
vexide:feat/vex-v5-thumb2

Conversation

@lewisfm

@lewisfmlewisfm commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

This PR switches the default codegen for the VEX V5 target to emit Thumb-2 instructions, allowing for smaller binary sizes (on the programs I tested this change on, I saw a ~20% size decrease on average). The target is renamed to have the thumb prefix instead of the arm prefix because of the updated instruction set.

Since VEXos starts all programs in Arm32 mode, the program entrypoint is explicitly compiled as ARM code and now transitions to Thumb mode when calling _start.

Users can still use the updated target in Arm32 mode by specifying -Ctarget-feature=-thumb-mode.

cc @tropicaaal@Gavin-Niederman

@rustbot

Copy link
Copy Markdown
Collaborator

Some changes occurred in src/doc/rustc/src/platform-support

cc @Noratrieb

These commits modify compiler targets.
(See the Target Tier Policy.)

@rustbotrustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Jul 4, 2026
@rustbot

Copy link
Copy Markdown
Collaborator

r? @Kivooeo

rustbot has assigned @Kivooeo.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: @ehuss, compiler
  • @ehuss, compiler expanded to 76 candidates
  • Random selection from 23 candidates

@rust-log-analyzer

Copy link
Copy Markdown
Collaborator

A job failed! Check out the build log: (web)(plain enhanced)(plain)

Click to see the possible cause of the failure (guessed by this bot)
 PR_CI_JOB: 1
IMAGE: x86_64-gnu-llvm-21
##[endgroup]
Updating crates.io index
error: failed to get `fnv` as a dependency of package `http v1.2.0`
... which satisfies dependency `http = "^1.1.0"` of package `ureq-proto v0.3.3`
... which satisfies dependency `ureq-proto = "^0.3.3"` of package `ureq v3.0.8`
... which satisfies dependency `ureq = "^3"` of package `citool v0.1.0 (/home/runner/work/rust/rust/src/ci/citool)`
Caused by:
failed to load source for dependency `fnv`
Caused by:
unable to update registry `crates-io`
Caused by:
download of 3/f/fnv failed
Caused by:
curl failed
Caused by:

@Gavin-NiedermanGavin-Niederman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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


**Tier: 3**

Allows compiling user programs for the [VEX V5 Brain](https://www.vexrobotics.com/276-4810.html), a microcontroller for educational and competitive robotics.

Rust support for this target is not affiliated with VEX Robotics or IFI, and does not link against any official VEX SDK.

This target was previously named `armv7a-vex-v5`.

@tropicaaaltropicaaalJul 11, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Not a blocker, but it might be a good idea for this to be elaborated on wrt. why we changed to thumb.

View changes since the review

@tropicaaaltropicaaal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@Kivooeo

Kivooeo commented Aug 5, 2026

Copy link
Copy Markdown
Member

@rustbot reroll (don't know anything about this targets and don't have bandwidth to figure it out)

@rustbotrustbot assigned jieyouxu and unassigned KivooeoAug 5, 2026
@jieyouxujieyouxu closed this Aug 5, 2026
@rustbotrustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 5, 2026
@jieyouxujieyouxu reopened this Aug 5, 2026
@rustbotrustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 5, 2026
@rustbot

Copy link
Copy Markdown
Collaborator

⚠️Warning⚠️

@jieyouxujieyouxu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks. Given that this is a Tier 3 target and target maintainers are on board I'm happy to stamp this.

@bors r+ rollup

View changes since this review

@rust-bors

rust-borsBot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 383d557 has been approved by jieyouxu

It is now in the queue for this repository.

@rust-borsrust-borsBot added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 5, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Aug 5, 2026
Emit thumb code on VEX V5
This PR switches the default codegen for the VEX V5 target to emit Thumb-2 instructions, allowing for smaller binary sizes (on the programs I tested this change on, I saw a ~20% size decrease on average). The target is renamed to have the `thumb` prefix instead of the `arm` prefix because of the updated instruction set.
Since VEXos starts all programs in Arm32 mode, the program entrypoint is explicitly compiled as ARM code and now transitions to Thumb mode when calling `_start`.
Users can still use the updated target in Arm32 mode by specifying `-Ctarget-feature=-thumb-mode`.
cc @tropicaaal@Gavin-Niederman
rust-borsBot pushed a commit that referenced this pull request Aug 5, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- #154585 (treat no_mangle_generic_items as hard error instead of lint warning)
- #158147 (std: fix stack buffer overflow in Windows junction_point)
- #160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- #160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- #160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- #160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- #160422 (move mir-opt miri tests to CI logic)
- #160444 (Avoid resolving path keywords outside `TypeNS`)
- #160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe my unsafe speculative flag)
- #155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- #158726 (std: move futex implementations into sys::sync::futex)
- #158762 (Emit thumb code on VEX V5)
- #159225 (Split IncrCompSession out of Session)
- #159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- #160198 (Rework `smallest_range_containing` to handle duplicates)
- #160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- #160390 (autodiff: Handle slice-tailed DSTs in type trees)
- #160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- #160516 (Add regression test for HRTB projection in closure)
- #160520 (Add some tests for specialization)
- #160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- #160523 (Add regression test for opaque type)
- #160531 (docs: fix typo in AllowExprMetavar comment)
- #160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- #160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
rust-borsBot pushed a commit that referenced this pull request Aug 5, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- #158147 (std: fix stack buffer overflow in Windows junction_point)
- #160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- #160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- #160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- #160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- #160422 (move mir-opt miri tests to CI logic)
- #160444 (Avoid resolving path keywords outside `TypeNS`)
- #160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe by unsafe speculative flag)
- #155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- #158726 (std: move futex implementations into sys::sync::futex)
- #158762 (Emit thumb code on VEX V5)
- #159225 (Split IncrCompSession out of Session)
- #159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- #160198 (Rework `smallest_range_containing` to handle duplicates)
- #160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- #160390 (autodiff: Handle slice-tailed DSTs in type trees)
- #160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- #160501 (Add bootstrap CLI snapshot test for testing miri)
- #160516 (Add regression test for HRTB projection in closure)
- #160520 (Add some tests for specialization)
- #160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- #160523 (Add regression test for opaque type)
- #160531 (docs: fix typo in AllowExprMetavar comment)
- #160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- #160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
@rust-bors
rust-borsBot merged commit 8c6b914 into rust-lang:mainAug 5, 2026
15 of 26 checks passed
@rustbotrustbot added this to the 1.99.0 milestone Aug 5, 2026
rust-timer added a commit that referenced this pull request Aug 5, 2026
Rollup merge of #158762 - vexide:feat/vex-v5-thumb2, r=jieyouxu
Emit thumb code on VEX V5
This PR switches the default codegen for the VEX V5 target to emit Thumb-2 instructions, allowing for smaller binary sizes (on the programs I tested this change on, I saw a ~20% size decrease on average). The target is renamed to have the `thumb` prefix instead of the `arm` prefix because of the updated instruction set.
Since VEXos starts all programs in Arm32 mode, the program entrypoint is explicitly compiled as ARM code and now transitions to Thumb mode when calling `_start`.
Users can still use the updated target in Arm32 mode by specifying `-Ctarget-feature=-thumb-mode`.
cc @tropicaaal@Gavin-Niederman
github-actionsBot pushed a commit to rust-lang/rustc-dev-guide that referenced this pull request Aug 10, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- rust-lang/rust#158147 (std: fix stack buffer overflow in Windows junction_point)
- rust-lang/rust#160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- rust-lang/rust#160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- rust-lang/rust#160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- rust-lang/rust#160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- rust-lang/rust#160422 (move mir-opt miri tests to CI logic)
- rust-lang/rust#160444 (Avoid resolving path keywords outside `TypeNS`)
- rust-lang/rust#160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe by unsafe speculative flag)
- rust-lang/rust#155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- rust-lang/rust#158726 (std: move futex implementations into sys::sync::futex)
- rust-lang/rust#158762 (Emit thumb code on VEX V5)
- rust-lang/rust#159225 (Split IncrCompSession out of Session)
- rust-lang/rust#159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- rust-lang/rust#160198 (Rework `smallest_range_containing` to handle duplicates)
- rust-lang/rust#160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- rust-lang/rust#160390 (autodiff: Handle slice-tailed DSTs in type trees)
- rust-lang/rust#160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- rust-lang/rust#160501 (Add bootstrap CLI snapshot test for testing miri)
- rust-lang/rust#160516 (Add regression test for HRTB projection in closure)
- rust-lang/rust#160520 (Add some tests for specialization)
- rust-lang/rust#160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- rust-lang/rust#160523 (Add regression test for opaque type)
- rust-lang/rust#160531 (docs: fix typo in AllowExprMetavar comment)
- rust-lang/rust#160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- rust-lang/rust#160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
flip1995 pushed a commit to flip1995/rust-clippy that referenced this pull request Aug 17, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- rust-lang/rust#158147 (std: fix stack buffer overflow in Windows junction_point)
- rust-lang/rust#160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- rust-lang/rust#160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- rust-lang/rust#160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- rust-lang/rust#160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- rust-lang/rust#160422 (move mir-opt miri tests to CI logic)
- rust-lang/rust#160444 (Avoid resolving path keywords outside `TypeNS`)
- rust-lang/rust#160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe by unsafe speculative flag)
- rust-lang/rust#155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- rust-lang/rust#158726 (std: move futex implementations into sys::sync::futex)
- rust-lang/rust#158762 (Emit thumb code on VEX V5)
- rust-lang/rust#159225 (Split IncrCompSession out of Session)
- rust-lang/rust#159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- rust-lang/rust#160198 (Rework `smallest_range_containing` to handle duplicates)
- rust-lang/rust#160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- rust-lang/rust#160390 (autodiff: Handle slice-tailed DSTs in type trees)
- rust-lang/rust#160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- rust-lang/rust#160501 (Add bootstrap CLI snapshot test for testing miri)
- rust-lang/rust#160516 (Add regression test for HRTB projection in closure)
- rust-lang/rust#160520 (Add some tests for specialization)
- rust-lang/rust#160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- rust-lang/rust#160523 (Add regression test for opaque type)
- rust-lang/rust#160531 (docs: fix typo in AllowExprMetavar comment)
- rust-lang/rust#160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- rust-lang/rust#160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.T-libsRelevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@lewisfm@rustbot@rust-log-analyzer@Kivooeo@jieyouxu@tropicaaal@Gavin-Niederman
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links"); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + '
Skip to content

Emit thumb code on VEX V5 - #158762

Merged
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
vexide:feat/vex-v5-thumb2
Aug 5, 2026
Merged

Emit thumb code on VEX V5#158762
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
vexide:feat/vex-v5-thumb2

Conversation

@lewisfm

@lewisfmlewisfm commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

This PR switches the default codegen for the VEX V5 target to emit Thumb-2 instructions, allowing for smaller binary sizes (on the programs I tested this change on, I saw a ~20% size decrease on average). The target is renamed to have the thumb prefix instead of the arm prefix because of the updated instruction set.

Since VEXos starts all programs in Arm32 mode, the program entrypoint is explicitly compiled as ARM code and now transitions to Thumb mode when calling _start.

Users can still use the updated target in Arm32 mode by specifying -Ctarget-feature=-thumb-mode.

cc @tropicaaal@Gavin-Niederman

@rustbot

Copy link
Copy Markdown
Collaborator

Some changes occurred in src/doc/rustc/src/platform-support

cc @Noratrieb

These commits modify compiler targets.
(See the Target Tier Policy.)

@rustbotrustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Jul 4, 2026
@rustbot

Copy link
Copy Markdown
Collaborator

r? @Kivooeo

rustbot has assigned @Kivooeo.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: @ehuss, compiler
  • @ehuss, compiler expanded to 76 candidates
  • Random selection from 23 candidates

@rust-log-analyzer

Copy link
Copy Markdown
Collaborator

A job failed! Check out the build log: (web)(plain enhanced)(plain)

Click to see the possible cause of the failure (guessed by this bot)
 PR_CI_JOB: 1
IMAGE: x86_64-gnu-llvm-21
##[endgroup]
Updating crates.io index
error: failed to get `fnv` as a dependency of package `http v1.2.0`
... which satisfies dependency `http = "^1.1.0"` of package `ureq-proto v0.3.3`
... which satisfies dependency `ureq-proto = "^0.3.3"` of package `ureq v3.0.8`
... which satisfies dependency `ureq = "^3"` of package `citool v0.1.0 (/home/runner/work/rust/rust/src/ci/citool)`
Caused by:
failed to load source for dependency `fnv`
Caused by:
unable to update registry `crates-io`
Caused by:
download of 3/f/fnv failed
Caused by:
curl failed
Caused by:

@Gavin-NiedermanGavin-Niederman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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


**Tier: 3**

Allows compiling user programs for the [VEX V5 Brain](https://www.vexrobotics.com/276-4810.html), a microcontroller for educational and competitive robotics.

Rust support for this target is not affiliated with VEX Robotics or IFI, and does not link against any official VEX SDK.

This target was previously named `armv7a-vex-v5`.

@tropicaaaltropicaaalJul 11, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Not a blocker, but it might be a good idea for this to be elaborated on wrt. why we changed to thumb.

View changes since the review

@tropicaaaltropicaaal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@Kivooeo

Kivooeo commented Aug 5, 2026

Copy link
Copy Markdown
Member

@rustbot reroll (don't know anything about this targets and don't have bandwidth to figure it out)

@rustbotrustbot assigned jieyouxu and unassigned KivooeoAug 5, 2026
@jieyouxujieyouxu closed this Aug 5, 2026
@rustbotrustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 5, 2026
@jieyouxujieyouxu reopened this Aug 5, 2026
@rustbotrustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 5, 2026
@rustbot

Copy link
Copy Markdown
Collaborator

⚠️Warning⚠️

@jieyouxujieyouxu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks. Given that this is a Tier 3 target and target maintainers are on board I'm happy to stamp this.

@bors r+ rollup

View changes since this review

@rust-bors

rust-borsBot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 383d557 has been approved by jieyouxu

It is now in the queue for this repository.

@rust-borsrust-borsBot added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 5, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Aug 5, 2026
Emit thumb code on VEX V5
This PR switches the default codegen for the VEX V5 target to emit Thumb-2 instructions, allowing for smaller binary sizes (on the programs I tested this change on, I saw a ~20% size decrease on average). The target is renamed to have the `thumb` prefix instead of the `arm` prefix because of the updated instruction set.
Since VEXos starts all programs in Arm32 mode, the program entrypoint is explicitly compiled as ARM code and now transitions to Thumb mode when calling `_start`.
Users can still use the updated target in Arm32 mode by specifying `-Ctarget-feature=-thumb-mode`.
cc @tropicaaal@Gavin-Niederman
rust-borsBot pushed a commit that referenced this pull request Aug 5, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- #154585 (treat no_mangle_generic_items as hard error instead of lint warning)
- #158147 (std: fix stack buffer overflow in Windows junction_point)
- #160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- #160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- #160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- #160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- #160422 (move mir-opt miri tests to CI logic)
- #160444 (Avoid resolving path keywords outside `TypeNS`)
- #160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe my unsafe speculative flag)
- #155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- #158726 (std: move futex implementations into sys::sync::futex)
- #158762 (Emit thumb code on VEX V5)
- #159225 (Split IncrCompSession out of Session)
- #159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- #160198 (Rework `smallest_range_containing` to handle duplicates)
- #160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- #160390 (autodiff: Handle slice-tailed DSTs in type trees)
- #160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- #160516 (Add regression test for HRTB projection in closure)
- #160520 (Add some tests for specialization)
- #160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- #160523 (Add regression test for opaque type)
- #160531 (docs: fix typo in AllowExprMetavar comment)
- #160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- #160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
rust-borsBot pushed a commit that referenced this pull request Aug 5, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- #158147 (std: fix stack buffer overflow in Windows junction_point)
- #160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- #160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- #160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- #160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- #160422 (move mir-opt miri tests to CI logic)
- #160444 (Avoid resolving path keywords outside `TypeNS`)
- #160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe by unsafe speculative flag)
- #155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- #158726 (std: move futex implementations into sys::sync::futex)
- #158762 (Emit thumb code on VEX V5)
- #159225 (Split IncrCompSession out of Session)
- #159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- #160198 (Rework `smallest_range_containing` to handle duplicates)
- #160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- #160390 (autodiff: Handle slice-tailed DSTs in type trees)
- #160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- #160501 (Add bootstrap CLI snapshot test for testing miri)
- #160516 (Add regression test for HRTB projection in closure)
- #160520 (Add some tests for specialization)
- #160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- #160523 (Add regression test for opaque type)
- #160531 (docs: fix typo in AllowExprMetavar comment)
- #160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- #160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
@rust-bors
rust-borsBot merged commit 8c6b914 into rust-lang:mainAug 5, 2026
15 of 26 checks passed
@rustbotrustbot added this to the 1.99.0 milestone Aug 5, 2026
rust-timer added a commit that referenced this pull request Aug 5, 2026
Rollup merge of #158762 - vexide:feat/vex-v5-thumb2, r=jieyouxu
Emit thumb code on VEX V5
This PR switches the default codegen for the VEX V5 target to emit Thumb-2 instructions, allowing for smaller binary sizes (on the programs I tested this change on, I saw a ~20% size decrease on average). The target is renamed to have the `thumb` prefix instead of the `arm` prefix because of the updated instruction set.
Since VEXos starts all programs in Arm32 mode, the program entrypoint is explicitly compiled as ARM code and now transitions to Thumb mode when calling `_start`.
Users can still use the updated target in Arm32 mode by specifying `-Ctarget-feature=-thumb-mode`.
cc @tropicaaal@Gavin-Niederman
github-actionsBot pushed a commit to rust-lang/rustc-dev-guide that referenced this pull request Aug 10, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- rust-lang/rust#158147 (std: fix stack buffer overflow in Windows junction_point)
- rust-lang/rust#160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- rust-lang/rust#160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- rust-lang/rust#160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- rust-lang/rust#160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- rust-lang/rust#160422 (move mir-opt miri tests to CI logic)
- rust-lang/rust#160444 (Avoid resolving path keywords outside `TypeNS`)
- rust-lang/rust#160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe by unsafe speculative flag)
- rust-lang/rust#155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- rust-lang/rust#158726 (std: move futex implementations into sys::sync::futex)
- rust-lang/rust#158762 (Emit thumb code on VEX V5)
- rust-lang/rust#159225 (Split IncrCompSession out of Session)
- rust-lang/rust#159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- rust-lang/rust#160198 (Rework `smallest_range_containing` to handle duplicates)
- rust-lang/rust#160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- rust-lang/rust#160390 (autodiff: Handle slice-tailed DSTs in type trees)
- rust-lang/rust#160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- rust-lang/rust#160501 (Add bootstrap CLI snapshot test for testing miri)
- rust-lang/rust#160516 (Add regression test for HRTB projection in closure)
- rust-lang/rust#160520 (Add some tests for specialization)
- rust-lang/rust#160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- rust-lang/rust#160523 (Add regression test for opaque type)
- rust-lang/rust#160531 (docs: fix typo in AllowExprMetavar comment)
- rust-lang/rust#160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- rust-lang/rust#160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
flip1995 pushed a commit to flip1995/rust-clippy that referenced this pull request Aug 17, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- rust-lang/rust#158147 (std: fix stack buffer overflow in Windows junction_point)
- rust-lang/rust#160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- rust-lang/rust#160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- rust-lang/rust#160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- rust-lang/rust#160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- rust-lang/rust#160422 (move mir-opt miri tests to CI logic)
- rust-lang/rust#160444 (Avoid resolving path keywords outside `TypeNS`)
- rust-lang/rust#160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe by unsafe speculative flag)
- rust-lang/rust#155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- rust-lang/rust#158726 (std: move futex implementations into sys::sync::futex)
- rust-lang/rust#158762 (Emit thumb code on VEX V5)
- rust-lang/rust#159225 (Split IncrCompSession out of Session)
- rust-lang/rust#159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- rust-lang/rust#160198 (Rework `smallest_range_containing` to handle duplicates)
- rust-lang/rust#160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- rust-lang/rust#160390 (autodiff: Handle slice-tailed DSTs in type trees)
- rust-lang/rust#160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- rust-lang/rust#160501 (Add bootstrap CLI snapshot test for testing miri)
- rust-lang/rust#160516 (Add regression test for HRTB projection in closure)
- rust-lang/rust#160520 (Add some tests for specialization)
- rust-lang/rust#160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- rust-lang/rust#160523 (Add regression test for opaque type)
- rust-lang/rust#160531 (docs: fix typo in AllowExprMetavar comment)
- rust-lang/rust#160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- rust-lang/rust#160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.T-libsRelevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@lewisfm@rustbot@rust-log-analyzer@Kivooeo@jieyouxu@tropicaaal@Gavin-Niederman
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default"); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

Emit thumb code on VEX V5 - #158762

Merged
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
vexide:feat/vex-v5-thumb2
Aug 5, 2026
Merged

Emit thumb code on VEX V5#158762
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
vexide:feat/vex-v5-thumb2

Conversation

@lewisfm

@lewisfmlewisfm commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

This PR switches the default codegen for the VEX V5 target to emit Thumb-2 instructions, allowing for smaller binary sizes (on the programs I tested this change on, I saw a ~20% size decrease on average). The target is renamed to have the thumb prefix instead of the arm prefix because of the updated instruction set.

Since VEXos starts all programs in Arm32 mode, the program entrypoint is explicitly compiled as ARM code and now transitions to Thumb mode when calling _start.

Users can still use the updated target in Arm32 mode by specifying -Ctarget-feature=-thumb-mode.

cc @tropicaaal@Gavin-Niederman

@rustbot

Copy link
Copy Markdown
Collaborator

Some changes occurred in src/doc/rustc/src/platform-support

cc @Noratrieb

These commits modify compiler targets.
(See the Target Tier Policy.)

@rustbotrustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Jul 4, 2026
@rustbot

Copy link
Copy Markdown
Collaborator

r? @Kivooeo

rustbot has assigned @Kivooeo.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: @ehuss, compiler
  • @ehuss, compiler expanded to 76 candidates
  • Random selection from 23 candidates

@rust-log-analyzer

Copy link
Copy Markdown
Collaborator

A job failed! Check out the build log: (web)(plain enhanced)(plain)

Click to see the possible cause of the failure (guessed by this bot)
 PR_CI_JOB: 1
IMAGE: x86_64-gnu-llvm-21
##[endgroup]
Updating crates.io index
error: failed to get `fnv` as a dependency of package `http v1.2.0`
... which satisfies dependency `http = "^1.1.0"` of package `ureq-proto v0.3.3`
... which satisfies dependency `ureq-proto = "^0.3.3"` of package `ureq v3.0.8`
... which satisfies dependency `ureq = "^3"` of package `citool v0.1.0 (/home/runner/work/rust/rust/src/ci/citool)`
Caused by:
failed to load source for dependency `fnv`
Caused by:
unable to update registry `crates-io`
Caused by:
download of 3/f/fnv failed
Caused by:
curl failed
Caused by:

@Gavin-NiedermanGavin-Niederman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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


**Tier: 3**

Allows compiling user programs for the [VEX V5 Brain](https://www.vexrobotics.com/276-4810.html), a microcontroller for educational and competitive robotics.

Rust support for this target is not affiliated with VEX Robotics or IFI, and does not link against any official VEX SDK.

This target was previously named `armv7a-vex-v5`.

@tropicaaaltropicaaalJul 11, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Not a blocker, but it might be a good idea for this to be elaborated on wrt. why we changed to thumb.

View changes since the review

@tropicaaaltropicaaal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@Kivooeo

Kivooeo commented Aug 5, 2026

Copy link
Copy Markdown
Member

@rustbot reroll (don't know anything about this targets and don't have bandwidth to figure it out)

@rustbotrustbot assigned jieyouxu and unassigned KivooeoAug 5, 2026
@jieyouxujieyouxu closed this Aug 5, 2026
@rustbotrustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 5, 2026
@jieyouxujieyouxu reopened this Aug 5, 2026
@rustbotrustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 5, 2026
@rustbot

Copy link
Copy Markdown
Collaborator

⚠️Warning⚠️

@jieyouxujieyouxu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks. Given that this is a Tier 3 target and target maintainers are on board I'm happy to stamp this.

@bors r+ rollup

View changes since this review

@rust-bors

rust-borsBot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 383d557 has been approved by jieyouxu

It is now in the queue for this repository.

@rust-borsrust-borsBot added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 5, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Aug 5, 2026
Emit thumb code on VEX V5
This PR switches the default codegen for the VEX V5 target to emit Thumb-2 instructions, allowing for smaller binary sizes (on the programs I tested this change on, I saw a ~20% size decrease on average). The target is renamed to have the `thumb` prefix instead of the `arm` prefix because of the updated instruction set.
Since VEXos starts all programs in Arm32 mode, the program entrypoint is explicitly compiled as ARM code and now transitions to Thumb mode when calling `_start`.
Users can still use the updated target in Arm32 mode by specifying `-Ctarget-feature=-thumb-mode`.
cc @tropicaaal@Gavin-Niederman
rust-borsBot pushed a commit that referenced this pull request Aug 5, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- #154585 (treat no_mangle_generic_items as hard error instead of lint warning)
- #158147 (std: fix stack buffer overflow in Windows junction_point)
- #160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- #160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- #160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- #160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- #160422 (move mir-opt miri tests to CI logic)
- #160444 (Avoid resolving path keywords outside `TypeNS`)
- #160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe my unsafe speculative flag)
- #155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- #158726 (std: move futex implementations into sys::sync::futex)
- #158762 (Emit thumb code on VEX V5)
- #159225 (Split IncrCompSession out of Session)
- #159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- #160198 (Rework `smallest_range_containing` to handle duplicates)
- #160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- #160390 (autodiff: Handle slice-tailed DSTs in type trees)
- #160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- #160516 (Add regression test for HRTB projection in closure)
- #160520 (Add some tests for specialization)
- #160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- #160523 (Add regression test for opaque type)
- #160531 (docs: fix typo in AllowExprMetavar comment)
- #160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- #160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
rust-borsBot pushed a commit that referenced this pull request Aug 5, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- #158147 (std: fix stack buffer overflow in Windows junction_point)
- #160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- #160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- #160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- #160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- #160422 (move mir-opt miri tests to CI logic)
- #160444 (Avoid resolving path keywords outside `TypeNS`)
- #160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe by unsafe speculative flag)
- #155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- #158726 (std: move futex implementations into sys::sync::futex)
- #158762 (Emit thumb code on VEX V5)
- #159225 (Split IncrCompSession out of Session)
- #159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- #160198 (Rework `smallest_range_containing` to handle duplicates)
- #160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- #160390 (autodiff: Handle slice-tailed DSTs in type trees)
- #160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- #160501 (Add bootstrap CLI snapshot test for testing miri)
- #160516 (Add regression test for HRTB projection in closure)
- #160520 (Add some tests for specialization)
- #160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- #160523 (Add regression test for opaque type)
- #160531 (docs: fix typo in AllowExprMetavar comment)
- #160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- #160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
@rust-bors
rust-borsBot merged commit 8c6b914 into rust-lang:mainAug 5, 2026
15 of 26 checks passed
@rustbotrustbot added this to the 1.99.0 milestone Aug 5, 2026
rust-timer added a commit that referenced this pull request Aug 5, 2026
Rollup merge of #158762 - vexide:feat/vex-v5-thumb2, r=jieyouxu
Emit thumb code on VEX V5
This PR switches the default codegen for the VEX V5 target to emit Thumb-2 instructions, allowing for smaller binary sizes (on the programs I tested this change on, I saw a ~20% size decrease on average). The target is renamed to have the `thumb` prefix instead of the `arm` prefix because of the updated instruction set.
Since VEXos starts all programs in Arm32 mode, the program entrypoint is explicitly compiled as ARM code and now transitions to Thumb mode when calling `_start`.
Users can still use the updated target in Arm32 mode by specifying `-Ctarget-feature=-thumb-mode`.
cc @tropicaaal@Gavin-Niederman
github-actionsBot pushed a commit to rust-lang/rustc-dev-guide that referenced this pull request Aug 10, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- rust-lang/rust#158147 (std: fix stack buffer overflow in Windows junction_point)
- rust-lang/rust#160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- rust-lang/rust#160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- rust-lang/rust#160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- rust-lang/rust#160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- rust-lang/rust#160422 (move mir-opt miri tests to CI logic)
- rust-lang/rust#160444 (Avoid resolving path keywords outside `TypeNS`)
- rust-lang/rust#160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe by unsafe speculative flag)
- rust-lang/rust#155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- rust-lang/rust#158726 (std: move futex implementations into sys::sync::futex)
- rust-lang/rust#158762 (Emit thumb code on VEX V5)
- rust-lang/rust#159225 (Split IncrCompSession out of Session)
- rust-lang/rust#159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- rust-lang/rust#160198 (Rework `smallest_range_containing` to handle duplicates)
- rust-lang/rust#160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- rust-lang/rust#160390 (autodiff: Handle slice-tailed DSTs in type trees)
- rust-lang/rust#160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- rust-lang/rust#160501 (Add bootstrap CLI snapshot test for testing miri)
- rust-lang/rust#160516 (Add regression test for HRTB projection in closure)
- rust-lang/rust#160520 (Add some tests for specialization)
- rust-lang/rust#160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- rust-lang/rust#160523 (Add regression test for opaque type)
- rust-lang/rust#160531 (docs: fix typo in AllowExprMetavar comment)
- rust-lang/rust#160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- rust-lang/rust#160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
flip1995 pushed a commit to flip1995/rust-clippy that referenced this pull request Aug 17, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- rust-lang/rust#158147 (std: fix stack buffer overflow in Windows junction_point)
- rust-lang/rust#160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- rust-lang/rust#160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- rust-lang/rust#160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- rust-lang/rust#160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- rust-lang/rust#160422 (move mir-opt miri tests to CI logic)
- rust-lang/rust#160444 (Avoid resolving path keywords outside `TypeNS`)
- rust-lang/rust#160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe by unsafe speculative flag)
- rust-lang/rust#155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- rust-lang/rust#158726 (std: move futex implementations into sys::sync::futex)
- rust-lang/rust#158762 (Emit thumb code on VEX V5)
- rust-lang/rust#159225 (Split IncrCompSession out of Session)
- rust-lang/rust#159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- rust-lang/rust#160198 (Rework `smallest_range_containing` to handle duplicates)
- rust-lang/rust#160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- rust-lang/rust#160390 (autodiff: Handle slice-tailed DSTs in type trees)
- rust-lang/rust#160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- rust-lang/rust#160501 (Add bootstrap CLI snapshot test for testing miri)
- rust-lang/rust#160516 (Add regression test for HRTB projection in closure)
- rust-lang/rust#160520 (Add some tests for specialization)
- rust-lang/rust#160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- rust-lang/rust#160523 (Add regression test for opaque type)
- rust-lang/rust#160531 (docs: fix typo in AllowExprMetavar comment)
- rust-lang/rust#160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- rust-lang/rust#160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.T-libsRelevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@lewisfm@rustbot@rust-log-analyzer@Kivooeo@jieyouxu@tropicaaal@Gavin-Niederman
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers"); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

Emit thumb code on VEX V5 - #158762

Merged
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
vexide:feat/vex-v5-thumb2
Aug 5, 2026
Merged

Emit thumb code on VEX V5#158762
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
vexide:feat/vex-v5-thumb2

Conversation

@lewisfm

@lewisfmlewisfm commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

This PR switches the default codegen for the VEX V5 target to emit Thumb-2 instructions, allowing for smaller binary sizes (on the programs I tested this change on, I saw a ~20% size decrease on average). The target is renamed to have the thumb prefix instead of the arm prefix because of the updated instruction set.

Since VEXos starts all programs in Arm32 mode, the program entrypoint is explicitly compiled as ARM code and now transitions to Thumb mode when calling _start.

Users can still use the updated target in Arm32 mode by specifying -Ctarget-feature=-thumb-mode.

cc @tropicaaal@Gavin-Niederman

@rustbot

Copy link
Copy Markdown
Collaborator

Some changes occurred in src/doc/rustc/src/platform-support

cc @Noratrieb

These commits modify compiler targets.
(See the Target Tier Policy.)

@rustbotrustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Jul 4, 2026
@rustbot

Copy link
Copy Markdown
Collaborator

r? @Kivooeo

rustbot has assigned @Kivooeo.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: @ehuss, compiler
  • @ehuss, compiler expanded to 76 candidates
  • Random selection from 23 candidates

@rust-log-analyzer

Copy link
Copy Markdown
Collaborator

A job failed! Check out the build log: (web)(plain enhanced)(plain)

Click to see the possible cause of the failure (guessed by this bot)
 PR_CI_JOB: 1
IMAGE: x86_64-gnu-llvm-21
##[endgroup]
Updating crates.io index
error: failed to get `fnv` as a dependency of package `http v1.2.0`
... which satisfies dependency `http = "^1.1.0"` of package `ureq-proto v0.3.3`
... which satisfies dependency `ureq-proto = "^0.3.3"` of package `ureq v3.0.8`
... which satisfies dependency `ureq = "^3"` of package `citool v0.1.0 (/home/runner/work/rust/rust/src/ci/citool)`
Caused by:
failed to load source for dependency `fnv`
Caused by:
unable to update registry `crates-io`
Caused by:
download of 3/f/fnv failed
Caused by:
curl failed
Caused by:

@Gavin-NiedermanGavin-Niederman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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


**Tier: 3**

Allows compiling user programs for the [VEX V5 Brain](https://www.vexrobotics.com/276-4810.html), a microcontroller for educational and competitive robotics.

Rust support for this target is not affiliated with VEX Robotics or IFI, and does not link against any official VEX SDK.

This target was previously named `armv7a-vex-v5`.

@tropicaaaltropicaaalJul 11, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Not a blocker, but it might be a good idea for this to be elaborated on wrt. why we changed to thumb.

View changes since the review

@tropicaaaltropicaaal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@Kivooeo

Kivooeo commented Aug 5, 2026

Copy link
Copy Markdown
Member

@rustbot reroll (don't know anything about this targets and don't have bandwidth to figure it out)

@rustbotrustbot assigned jieyouxu and unassigned KivooeoAug 5, 2026
@jieyouxujieyouxu closed this Aug 5, 2026
@rustbotrustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 5, 2026
@jieyouxujieyouxu reopened this Aug 5, 2026
@rustbotrustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 5, 2026
@rustbot

Copy link
Copy Markdown
Collaborator

⚠️Warning⚠️

@jieyouxujieyouxu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks. Given that this is a Tier 3 target and target maintainers are on board I'm happy to stamp this.

@bors r+ rollup

View changes since this review

@rust-bors

rust-borsBot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 383d557 has been approved by jieyouxu

It is now in the queue for this repository.

@rust-borsrust-borsBot added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 5, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Aug 5, 2026
Emit thumb code on VEX V5
This PR switches the default codegen for the VEX V5 target to emit Thumb-2 instructions, allowing for smaller binary sizes (on the programs I tested this change on, I saw a ~20% size decrease on average). The target is renamed to have the `thumb` prefix instead of the `arm` prefix because of the updated instruction set.
Since VEXos starts all programs in Arm32 mode, the program entrypoint is explicitly compiled as ARM code and now transitions to Thumb mode when calling `_start`.
Users can still use the updated target in Arm32 mode by specifying `-Ctarget-feature=-thumb-mode`.
cc @tropicaaal@Gavin-Niederman
rust-borsBot pushed a commit that referenced this pull request Aug 5, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- #154585 (treat no_mangle_generic_items as hard error instead of lint warning)
- #158147 (std: fix stack buffer overflow in Windows junction_point)
- #160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- #160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- #160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- #160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- #160422 (move mir-opt miri tests to CI logic)
- #160444 (Avoid resolving path keywords outside `TypeNS`)
- #160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe my unsafe speculative flag)
- #155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- #158726 (std: move futex implementations into sys::sync::futex)
- #158762 (Emit thumb code on VEX V5)
- #159225 (Split IncrCompSession out of Session)
- #159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- #160198 (Rework `smallest_range_containing` to handle duplicates)
- #160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- #160390 (autodiff: Handle slice-tailed DSTs in type trees)
- #160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- #160516 (Add regression test for HRTB projection in closure)
- #160520 (Add some tests for specialization)
- #160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- #160523 (Add regression test for opaque type)
- #160531 (docs: fix typo in AllowExprMetavar comment)
- #160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- #160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
rust-borsBot pushed a commit that referenced this pull request Aug 5, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- #158147 (std: fix stack buffer overflow in Windows junction_point)
- #160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- #160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- #160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- #160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- #160422 (move mir-opt miri tests to CI logic)
- #160444 (Avoid resolving path keywords outside `TypeNS`)
- #160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe by unsafe speculative flag)
- #155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- #158726 (std: move futex implementations into sys::sync::futex)
- #158762 (Emit thumb code on VEX V5)
- #159225 (Split IncrCompSession out of Session)
- #159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- #160198 (Rework `smallest_range_containing` to handle duplicates)
- #160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- #160390 (autodiff: Handle slice-tailed DSTs in type trees)
- #160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- #160501 (Add bootstrap CLI snapshot test for testing miri)
- #160516 (Add regression test for HRTB projection in closure)
- #160520 (Add some tests for specialization)
- #160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- #160523 (Add regression test for opaque type)
- #160531 (docs: fix typo in AllowExprMetavar comment)
- #160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- #160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
@rust-bors
rust-borsBot merged commit 8c6b914 into rust-lang:mainAug 5, 2026
15 of 26 checks passed
@rustbotrustbot added this to the 1.99.0 milestone Aug 5, 2026
rust-timer added a commit that referenced this pull request Aug 5, 2026
Rollup merge of #158762 - vexide:feat/vex-v5-thumb2, r=jieyouxu
Emit thumb code on VEX V5
This PR switches the default codegen for the VEX V5 target to emit Thumb-2 instructions, allowing for smaller binary sizes (on the programs I tested this change on, I saw a ~20% size decrease on average). The target is renamed to have the `thumb` prefix instead of the `arm` prefix because of the updated instruction set.
Since VEXos starts all programs in Arm32 mode, the program entrypoint is explicitly compiled as ARM code and now transitions to Thumb mode when calling `_start`.
Users can still use the updated target in Arm32 mode by specifying `-Ctarget-feature=-thumb-mode`.
cc @tropicaaal@Gavin-Niederman
github-actionsBot pushed a commit to rust-lang/rustc-dev-guide that referenced this pull request Aug 10, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- rust-lang/rust#158147 (std: fix stack buffer overflow in Windows junction_point)
- rust-lang/rust#160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- rust-lang/rust#160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- rust-lang/rust#160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- rust-lang/rust#160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- rust-lang/rust#160422 (move mir-opt miri tests to CI logic)
- rust-lang/rust#160444 (Avoid resolving path keywords outside `TypeNS`)
- rust-lang/rust#160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe by unsafe speculative flag)
- rust-lang/rust#155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- rust-lang/rust#158726 (std: move futex implementations into sys::sync::futex)
- rust-lang/rust#158762 (Emit thumb code on VEX V5)
- rust-lang/rust#159225 (Split IncrCompSession out of Session)
- rust-lang/rust#159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- rust-lang/rust#160198 (Rework `smallest_range_containing` to handle duplicates)
- rust-lang/rust#160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- rust-lang/rust#160390 (autodiff: Handle slice-tailed DSTs in type trees)
- rust-lang/rust#160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- rust-lang/rust#160501 (Add bootstrap CLI snapshot test for testing miri)
- rust-lang/rust#160516 (Add regression test for HRTB projection in closure)
- rust-lang/rust#160520 (Add some tests for specialization)
- rust-lang/rust#160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- rust-lang/rust#160523 (Add regression test for opaque type)
- rust-lang/rust#160531 (docs: fix typo in AllowExprMetavar comment)
- rust-lang/rust#160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- rust-lang/rust#160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
flip1995 pushed a commit to flip1995/rust-clippy that referenced this pull request Aug 17, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- rust-lang/rust#158147 (std: fix stack buffer overflow in Windows junction_point)
- rust-lang/rust#160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- rust-lang/rust#160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- rust-lang/rust#160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- rust-lang/rust#160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- rust-lang/rust#160422 (move mir-opt miri tests to CI logic)
- rust-lang/rust#160444 (Avoid resolving path keywords outside `TypeNS`)
- rust-lang/rust#160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe by unsafe speculative flag)
- rust-lang/rust#155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- rust-lang/rust#158726 (std: move futex implementations into sys::sync::futex)
- rust-lang/rust#158762 (Emit thumb code on VEX V5)
- rust-lang/rust#159225 (Split IncrCompSession out of Session)
- rust-lang/rust#159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- rust-lang/rust#160198 (Rework `smallest_range_containing` to handle duplicates)
- rust-lang/rust#160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- rust-lang/rust#160390 (autodiff: Handle slice-tailed DSTs in type trees)
- rust-lang/rust#160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- rust-lang/rust#160501 (Add bootstrap CLI snapshot test for testing miri)
- rust-lang/rust#160516 (Add regression test for HRTB projection in closure)
- rust-lang/rust#160520 (Add some tests for specialization)
- rust-lang/rust#160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- rust-lang/rust#160523 (Add regression test for opaque type)
- rust-lang/rust#160531 (docs: fix typo in AllowExprMetavar comment)
- rust-lang/rust#160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- rust-lang/rust#160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.T-libsRelevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@lewisfm@rustbot@rust-log-analyzer@Kivooeo@jieyouxu@tropicaaal@Gavin-Niederman
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode"); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })();
Skip to content

Emit thumb code on VEX V5 - #158762

Merged
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
vexide:feat/vex-v5-thumb2
Aug 5, 2026
Merged

Emit thumb code on VEX V5#158762
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
vexide:feat/vex-v5-thumb2

Conversation

@lewisfm

@lewisfmlewisfm commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

This PR switches the default codegen for the VEX V5 target to emit Thumb-2 instructions, allowing for smaller binary sizes (on the programs I tested this change on, I saw a ~20% size decrease on average). The target is renamed to have the thumb prefix instead of the arm prefix because of the updated instruction set.

Since VEXos starts all programs in Arm32 mode, the program entrypoint is explicitly compiled as ARM code and now transitions to Thumb mode when calling _start.

Users can still use the updated target in Arm32 mode by specifying -Ctarget-feature=-thumb-mode.

cc @tropicaaal@Gavin-Niederman

@rustbot

Copy link
Copy Markdown
Collaborator

Some changes occurred in src/doc/rustc/src/platform-support

cc @Noratrieb

These commits modify compiler targets.
(See the Target Tier Policy.)

@rustbotrustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Jul 4, 2026
@rustbot

Copy link
Copy Markdown
Collaborator

r? @Kivooeo

rustbot has assigned @Kivooeo.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: @ehuss, compiler
  • @ehuss, compiler expanded to 76 candidates
  • Random selection from 23 candidates

@rust-log-analyzer

Copy link
Copy Markdown
Collaborator

A job failed! Check out the build log: (web)(plain enhanced)(plain)

Click to see the possible cause of the failure (guessed by this bot)
 PR_CI_JOB: 1
IMAGE: x86_64-gnu-llvm-21
##[endgroup]
Updating crates.io index
error: failed to get `fnv` as a dependency of package `http v1.2.0`
... which satisfies dependency `http = "^1.1.0"` of package `ureq-proto v0.3.3`
... which satisfies dependency `ureq-proto = "^0.3.3"` of package `ureq v3.0.8`
... which satisfies dependency `ureq = "^3"` of package `citool v0.1.0 (/home/runner/work/rust/rust/src/ci/citool)`
Caused by:
failed to load source for dependency `fnv`
Caused by:
unable to update registry `crates-io`
Caused by:
download of 3/f/fnv failed
Caused by:
curl failed
Caused by:

@Gavin-NiedermanGavin-Niederman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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


**Tier: 3**

Allows compiling user programs for the [VEX V5 Brain](https://www.vexrobotics.com/276-4810.html), a microcontroller for educational and competitive robotics.

Rust support for this target is not affiliated with VEX Robotics or IFI, and does not link against any official VEX SDK.

This target was previously named `armv7a-vex-v5`.

@tropicaaaltropicaaalJul 11, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Not a blocker, but it might be a good idea for this to be elaborated on wrt. why we changed to thumb.

View changes since the review

@tropicaaaltropicaaal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@Kivooeo

Kivooeo commented Aug 5, 2026

Copy link
Copy Markdown
Member

@rustbot reroll (don't know anything about this targets and don't have bandwidth to figure it out)

@rustbotrustbot assigned jieyouxu and unassigned KivooeoAug 5, 2026
@jieyouxujieyouxu closed this Aug 5, 2026
@rustbotrustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 5, 2026
@jieyouxujieyouxu reopened this Aug 5, 2026
@rustbotrustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 5, 2026
@rustbot

Copy link
Copy Markdown
Collaborator

⚠️Warning⚠️

@jieyouxujieyouxu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks. Given that this is a Tier 3 target and target maintainers are on board I'm happy to stamp this.

@bors r+ rollup

View changes since this review

@rust-bors

rust-borsBot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 383d557 has been approved by jieyouxu

It is now in the queue for this repository.

@rust-borsrust-borsBot added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 5, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Aug 5, 2026
Emit thumb code on VEX V5
This PR switches the default codegen for the VEX V5 target to emit Thumb-2 instructions, allowing for smaller binary sizes (on the programs I tested this change on, I saw a ~20% size decrease on average). The target is renamed to have the `thumb` prefix instead of the `arm` prefix because of the updated instruction set.
Since VEXos starts all programs in Arm32 mode, the program entrypoint is explicitly compiled as ARM code and now transitions to Thumb mode when calling `_start`.
Users can still use the updated target in Arm32 mode by specifying `-Ctarget-feature=-thumb-mode`.
cc @tropicaaal@Gavin-Niederman
rust-borsBot pushed a commit that referenced this pull request Aug 5, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- #154585 (treat no_mangle_generic_items as hard error instead of lint warning)
- #158147 (std: fix stack buffer overflow in Windows junction_point)
- #160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- #160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- #160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- #160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- #160422 (move mir-opt miri tests to CI logic)
- #160444 (Avoid resolving path keywords outside `TypeNS`)
- #160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe my unsafe speculative flag)
- #155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- #158726 (std: move futex implementations into sys::sync::futex)
- #158762 (Emit thumb code on VEX V5)
- #159225 (Split IncrCompSession out of Session)
- #159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- #160198 (Rework `smallest_range_containing` to handle duplicates)
- #160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- #160390 (autodiff: Handle slice-tailed DSTs in type trees)
- #160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- #160516 (Add regression test for HRTB projection in closure)
- #160520 (Add some tests for specialization)
- #160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- #160523 (Add regression test for opaque type)
- #160531 (docs: fix typo in AllowExprMetavar comment)
- #160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- #160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
rust-borsBot pushed a commit that referenced this pull request Aug 5, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- #158147 (std: fix stack buffer overflow in Windows junction_point)
- #160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- #160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- #160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- #160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- #160422 (move mir-opt miri tests to CI logic)
- #160444 (Avoid resolving path keywords outside `TypeNS`)
- #160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe by unsafe speculative flag)
- #155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- #158726 (std: move futex implementations into sys::sync::futex)
- #158762 (Emit thumb code on VEX V5)
- #159225 (Split IncrCompSession out of Session)
- #159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- #160198 (Rework `smallest_range_containing` to handle duplicates)
- #160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- #160390 (autodiff: Handle slice-tailed DSTs in type trees)
- #160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- #160501 (Add bootstrap CLI snapshot test for testing miri)
- #160516 (Add regression test for HRTB projection in closure)
- #160520 (Add some tests for specialization)
- #160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- #160523 (Add regression test for opaque type)
- #160531 (docs: fix typo in AllowExprMetavar comment)
- #160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- #160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
@rust-bors
rust-borsBot merged commit 8c6b914 into rust-lang:mainAug 5, 2026
15 of 26 checks passed
@rustbotrustbot added this to the 1.99.0 milestone Aug 5, 2026
rust-timer added a commit that referenced this pull request Aug 5, 2026
Rollup merge of #158762 - vexide:feat/vex-v5-thumb2, r=jieyouxu
Emit thumb code on VEX V5
This PR switches the default codegen for the VEX V5 target to emit Thumb-2 instructions, allowing for smaller binary sizes (on the programs I tested this change on, I saw a ~20% size decrease on average). The target is renamed to have the `thumb` prefix instead of the `arm` prefix because of the updated instruction set.
Since VEXos starts all programs in Arm32 mode, the program entrypoint is explicitly compiled as ARM code and now transitions to Thumb mode when calling `_start`.
Users can still use the updated target in Arm32 mode by specifying `-Ctarget-feature=-thumb-mode`.
cc @tropicaaal@Gavin-Niederman
github-actionsBot pushed a commit to rust-lang/rustc-dev-guide that referenced this pull request Aug 10, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- rust-lang/rust#158147 (std: fix stack buffer overflow in Windows junction_point)
- rust-lang/rust#160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- rust-lang/rust#160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- rust-lang/rust#160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- rust-lang/rust#160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- rust-lang/rust#160422 (move mir-opt miri tests to CI logic)
- rust-lang/rust#160444 (Avoid resolving path keywords outside `TypeNS`)
- rust-lang/rust#160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe by unsafe speculative flag)
- rust-lang/rust#155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- rust-lang/rust#158726 (std: move futex implementations into sys::sync::futex)
- rust-lang/rust#158762 (Emit thumb code on VEX V5)
- rust-lang/rust#159225 (Split IncrCompSession out of Session)
- rust-lang/rust#159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- rust-lang/rust#160198 (Rework `smallest_range_containing` to handle duplicates)
- rust-lang/rust#160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- rust-lang/rust#160390 (autodiff: Handle slice-tailed DSTs in type trees)
- rust-lang/rust#160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- rust-lang/rust#160501 (Add bootstrap CLI snapshot test for testing miri)
- rust-lang/rust#160516 (Add regression test for HRTB projection in closure)
- rust-lang/rust#160520 (Add some tests for specialization)
- rust-lang/rust#160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- rust-lang/rust#160523 (Add regression test for opaque type)
- rust-lang/rust#160531 (docs: fix typo in AllowExprMetavar comment)
- rust-lang/rust#160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- rust-lang/rust#160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
flip1995 pushed a commit to flip1995/rust-clippy that referenced this pull request Aug 17, 2026
…uwer
Rollup of 25 pull requests
Successful merges:
- rust-lang/rust#158147 (std: fix stack buffer overflow in Windows junction_point)
- rust-lang/rust#160130 (Select cache values to verify by key fingerprint, not value fingerprint)
- rust-lang/rust#160343 (Rename `OutlivesPredicate` to `OutlivesClause`)
- rust-lang/rust#160360 (Remove rustc_middle dependency on rustc_hir_pretty)
- rust-lang/rust#160387 (rustc_codegen_ssa: Correctly apply the static `--jobs-backend` limit to backend parallelism)
- rust-lang/rust#160422 (move mir-opt miri tests to CI logic)
- rust-lang/rust#160444 (Avoid resolving path keywords outside `TypeNS`)
- rust-lang/rust#160510 (Resolver: (un)tracked borrows for `CmRefCell` made safe by unsafe speculative flag)
- rust-lang/rust#155424 ([blocked] Link to proposed LLM policy in CONTRIBUTING and pull request template)
- rust-lang/rust#158726 (std: move futex implementations into sys::sync::futex)
- rust-lang/rust#158762 (Emit thumb code on VEX V5)
- rust-lang/rust#159225 (Split IncrCompSession out of Session)
- rust-lang/rust#159820 (Make the `rustc_unsafe_specialization_marker` attribute actually `unsafe`)
- rust-lang/rust#160198 (Rework `smallest_range_containing` to handle duplicates)
- rust-lang/rust#160362 (Split `SpannedTypeVisitor` into its own crate, `rustc_ty_walk`)
- rust-lang/rust#160390 (autodiff: Handle slice-tailed DSTs in type trees)
- rust-lang/rust#160420 (Suggest `cast_signed()` for overflowing signed integer literals)
- rust-lang/rust#160501 (Add bootstrap CLI snapshot test for testing miri)
- rust-lang/rust#160516 (Add regression test for HRTB projection in closure)
- rust-lang/rust#160520 (Add some tests for specialization)
- rust-lang/rust#160522 (fix(bootstrap): Normalize the names of proc macro dependency crates)
- rust-lang/rust#160523 (Add regression test for opaque type)
- rust-lang/rust#160531 (docs: fix typo in AllowExprMetavar comment)
- rust-lang/rust#160538 (Update expect messages in tcp.rs doc examples to follow the style guide)
- rust-lang/rust#160548 (bootstrap: Register `coverage-map` and `coverage-run` aliases via a separate step)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.T-libsRelevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@lewisfm@rustbot@rust-log-analyzer@Kivooeo@jieyouxu@tropicaaal@Gavin-Niederman