Fix autodiff_illegal.rs test - #160768

Merged
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
JonathanBrouwer:fix_autodiff_ci
Aug 9, 2026
Merged

Fix autodiff_illegal.rs test#160768
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
JonathanBrouwer:fix_autodiff_ci

Conversation

@JonathanBrouwer

@JonathanBrouwerJonathanBrouwer commented Aug 8, 2026•
edited
Loading

Copy link
Copy Markdown
Member

This fixes the current CI failure: #160761 (comment)

r? @ZuseZ4

  • I did not use an LLM to create a change in this PR.
  • I used an LLM to create a change in this PR, and I have explained below how it was used.

@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. labels Aug 8, 2026
mejrs
mejrs reviewed Aug 8, 2026
View reviewed changes

@mejrsmejrs left a comment •
edited by rustbot
Loading

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.

Looks like fallout of #160577

Is it correct to enable proc_macro_hygiene here rather than just change the test output? The test did not previously enable stmt_expr_attributes, for example.

Note also that proc_macro_hygiene is sort of a grab bag of things that macros can't be on and we're likely to move individual items to stable rather than stabilize the entire feature at once. As such if you want this test to reflect changes in what positions macros are allowed in it would be better to not enable the feature.

View changes since this review

@JonathanBrouwerJonathanBrouwer changed the title Add proc_macro_hygiene feature gate to autodiff_illegal.rs to fix CIFix autodiff_illegal.rs testAug 8, 2026
@JonathanBrouwer

JonathanBrouwer commented Aug 8, 2026

Copy link
Copy Markdown
MemberAuthor

@mejrs Agreed, fixed

@JonathanBrouwer

JonathanBrouwer commented Aug 8, 2026

Copy link
Copy Markdown
MemberAuthor

@bors try jobs=optional-x86_64-gnu-autodiff

@rust-bors

This comment has been minimized.

Sign in to view
rust-borsBot pushed a commit that referenced this pull request Aug 8, 2026
@bors
Auto merge of#160768- JonathanBrouwer:fix_autodiff_ci, r=<try>
dbd7f24
Fix autodiff_illegal.rs test
try-job: optional-x86_64-gnu-autodiff
mejrs
mejrs approved these changes Aug 8, 2026
View reviewed changes

@mejrsmejrs left a comment •
edited by rustbot
Loading

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.

r=me once green unless you want to wait for the autodiff guys to review, but best to merge this soon because the fail message is annoying me already

View changes since this review

@rust-bors

rust-borsBot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

☀️ Try build successful (CI)
Build commit: dbd7f24 (dbd7f2441d0c37ebd150ecacd40f4bf771c6e63b)
Base parent: 771916f (771916f9028e7fe56d2685f2c4f698de5d7d6a45)

@ZuseZ4

ZuseZ4 commented Aug 8, 2026

Copy link
Copy Markdown
Member

@bors r=mejrs,ZuseZ4 rollup

@rust-bors

rust-borsBot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

📌 Commit acd2f65 has been approved by mejrs,ZuseZ4

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 8, 2026
jhpratt added a commit to jhpratt/rust that referenced this pull request Aug 9, 2026
@jhpratt
Rollup merge ofrust-lang#160768- JonathanBrouwer:fix_autodiff_ci, r…
866f202
…=mejrs,ZuseZ4
Fix autodiff_illegal.rs test
This fixes the current CI failure: rust-lang#160761 (comment)
r? @ZuseZ4
@jhprattjhpratt mentioned this pull request Aug 9, 2026
Closed
rust-borsBot pushed a commit that referenced this pull request Aug 9, 2026
@bors
Auto merge of#160786- jhpratt:rollup-Mfq8vBr, r=jhpratt
74dbe2d
Rollup of 6 pull requests
Successful merges:
- #160529 (Upgrade and deduplicate dependencies)
- #158517 (Initial implementation of named `Fn` trait parameters)
- #158587 (check if len of array const arg matches the expected len of the type when lowering to valtree)
- #160748 (Add regression test for cross-crate assoc const private field leak in rustdoc)
- #160708 (remove old update mechanism)
- #160768 (Fix autodiff_illegal.rs test)
@jhprattjhpratt mentioned this pull request Aug 9, 2026
Merged
rust-borsBot pushed a commit that referenced this pull request Aug 9, 2026
@bors
Auto merge of#160789- jhpratt:rollup-zFeRr02, r=jhpratt
153ecc4
Rollup of 5 pull requests
Successful merges:
- #158517 (Initial implementation of named `Fn` trait parameters)
- #158587 (check if len of array const arg matches the expected len of the type when lowering to valtree)
- #160748 (Add regression test for cross-crate assoc const private field leak in rustdoc)
- #160708 (remove old update mechanism)
- #160768 (Fix autodiff_illegal.rs test)
@rust-bors
rust-borsBot merged commit f5c9a72 into rust-lang:mainAug 9, 2026
14 checks passed
@rustbotrustbot added this to the 1.99.0 milestone Aug 9, 2026
rust-timer added a commit that referenced this pull request Aug 9, 2026
@rust-timer
Unrolled build for#160768
b7148e0
Rollup merge of #160768 - JonathanBrouwer:fix_autodiff_ci, r=mejrs,ZuseZ4
Fix autodiff_illegal.rs test
This fixes the current CI failure: #160761 (comment)
r? @ZuseZ4
@cjgillotcjgillot mentioned this pull request Aug 9, 2026
Open
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mejrsmejrsmejrs approved these changes
@ZuseZ4ZuseZ4Awaiting requested review from ZuseZ4

Assignees

@ZuseZ4ZuseZ4

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.

Projects

None yet

Milestone

1.99.0

Development

Successfully merging this pull request may close these issues.

4 participants

@JonathanBrouwer@ZuseZ4@mejrs@rustbot
, 'i'); if (__m === '*' || __re.test(location.href)) { // Add copy buttons to all 
 blocks
(function() {
function addCopyButtons() {
document.querySelectorAll('pre code').forEach(function(codeBlock) {
if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;
codeBlock.parentElement.setAttribute('data-copy-added', 'true');
var btn = document.createElement('button');
btn.textContent = 'Copy';
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;';
btn.onmouseover = function() { this.style.opacity = '1'; };
btn.onmouseout = function() { this.style.opacity = '0.7'; };
btn.onclick = function() {
navigator.clipboard.writeText(codeBlock.textContent).then(function() {
btn.textContent = 'Copied!';
setTimeout(function() { btn.textContent = 'Copy'; }, 1500);
});
};
codeBlock.parentElement.style.position = 'relative';
codeBlock.parentElement.appendChild(btn);
});
}
addCopyButtons();
// Re-run on dynamic content
var observer = new MutationObserver(addCopyButtons);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} 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
Fix autodiff_illegal.rs test - #160768
Merged
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom 
JonathanBrouwer:fix_autodiff_ci
Aug 9, 2026
Merged
Fix autodiff_illegal.rs test#160768
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom 
JonathanBrouwer:fix_autodiff_ci
Conversation
@JonathanBrouwer
@JonathanBrouwerJonathanBrouwer
commented
Aug 8, 2026•

edited

Loading

Copy link


Copy Markdown

Member
This fixes the current CI failure: #160761 (comment)
r? @ZuseZ4
  •  I did not use an LLM to create a change in this PR.
  •  I used an LLM to create a change in this PR, and I have explained below how it was used.
@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.
labels
Aug 8, 2026
mejrs
mejrs
reviewed
Aug 8, 2026
View reviewed changes
@mejrsmejrs
left a comment
•

edited by rustbot

Loading

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.

Looks like fallout of #160577
Is it correct to enable proc_macro_hygiene here rather than just change the test output? The test did not previously enable stmt_expr_attributes, for example.
Note also that proc_macro_hygiene is sort of a grab bag of things that macros can't be on and we're likely to move individual items to stable rather than stabilize the entire feature at once. As such if you want this test to reflect changes in what positions macros are allowed in it would be better to not enable the feature.
View changes since this review
@JonathanBrouwerJonathanBrouwer
changed the title
Add proc_macro_hygiene feature gate to autodiff_illegal.rs to fix CIFix autodiff_illegal.rs testAug 8, 2026
@JonathanBrouwer
JonathanBrouwer
commented
Aug 8, 2026

Copy link


Copy Markdown

MemberAuthor
@mejrs Agreed, fixed
@JonathanBrouwer
JonathanBrouwer
commented
Aug 8, 2026

Copy link


Copy Markdown

MemberAuthor
@bors try jobs=optional-x86_64-gnu-autodiff
@rust-bors

This comment has been minimized.

Sign in to view
rust-borsBot
pushed a commit
that referenced
this pull request
Aug 8, 2026
@bors
Auto merge of#160768- JonathanBrouwer:fix_autodiff_ci, r=<try>
dbd7f24
Fix autodiff_illegal.rs test
try-job: optional-x86_64-gnu-autodiff
mejrs
mejrs
approved these changes
Aug 8, 2026
View reviewed changes
@mejrsmejrs
left a comment
•

edited by rustbot

Loading

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.

r=me once green unless you want to wait for the autodiff guys to review, but best to merge this soon because the fail message is annoying me already
View changes since this review
@rust-bors
rust-borsBot
commented
Aug 8, 2026

Copy link


Copy Markdown

Contributor
☀️ Try build successful (CI)

Build commit: dbd7f24 (dbd7f2441d0c37ebd150ecacd40f4bf771c6e63b)

Base parent: 771916f (771916f9028e7fe56d2685f2c4f698de5d7d6a45)
@ZuseZ4
ZuseZ4
commented
Aug 8, 2026

Copy link


Copy Markdown

Member
@bors r=mejrs,ZuseZ4 rollup
@rust-bors
rust-borsBot
commented
Aug 8, 2026

Copy link


Copy Markdown

Contributor
📌 Commit acd2f65 has been approved by mejrs,ZuseZ4
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 8, 2026
jhpratt added a commit
to jhpratt/rust
that referenced
this pull request
Aug 9, 2026
@jhpratt
Rollup merge ofrust-lang#160768- JonathanBrouwer:fix_autodiff_ci, r…
866f202
…=mejrs,ZuseZ4
Fix autodiff_illegal.rs test
This fixes the current CI failure: rust-lang#160761 (comment)
r? @ZuseZ4
@jhprattjhpratt
mentioned this pull request
Aug 9, 2026
 Closed

rust-borsBot
pushed a commit
that referenced
this pull request
Aug 9, 2026
@bors
Auto merge of#160786- jhpratt:rollup-Mfq8vBr, r=jhpratt
74dbe2d
Rollup of 6 pull requests
Successful merges:
- #160529 (Upgrade and deduplicate dependencies)
- #158517 (Initial implementation of named `Fn` trait parameters)
- #158587 (check if len of array const arg matches the expected len of the type when lowering to valtree)
- #160748 (Add regression test for cross-crate assoc const private field leak in rustdoc)
- #160708 (remove old update mechanism)
- #160768 (Fix autodiff_illegal.rs test)
@jhprattjhpratt
mentioned this pull request
Aug 9, 2026
 Merged

rust-borsBot
pushed a commit
that referenced
this pull request
Aug 9, 2026
@bors
Auto merge of#160789- jhpratt:rollup-zFeRr02, r=jhpratt
153ecc4
Rollup of 5 pull requests
Successful merges:
- #158517 (Initial implementation of named `Fn` trait parameters)
- #158587 (check if len of array const arg matches the expected len of the type when lowering to valtree)
- #160748 (Add regression test for cross-crate assoc const private field leak in rustdoc)
- #160708 (remove old update mechanism)
- #160768 (Fix autodiff_illegal.rs test)
@rust-bors
rust-borsBot
merged commit f5c9a72
into
rust-lang:mainAug 9, 2026

14 checks passed

@rustbotrustbot
added this to the 1.99.0 milestone
Aug 9, 2026
rust-timer added a commit
that referenced
this pull request
Aug 9, 2026
@rust-timer
Unrolled build for#160768
b7148e0
Rollup merge of #160768 - JonathanBrouwer:fix_autodiff_ci, r=mejrs,ZuseZ4
Fix autodiff_illegal.rs test
This fixes the current CI failure: #160761 (comment)
r? @ZuseZ4
@cjgillotcjgillot
mentioned this pull request
Aug 9, 2026
 Open

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@mejrsmejrsmejrs approved these changes
@ZuseZ4ZuseZ4Awaiting requested review from ZuseZ4

Assignees

@ZuseZ4ZuseZ4

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.

Projects


None yet


Milestone

1.99.0

Development

Successfully merging this pull request may close these issues.

4 participants

@JonathanBrouwer@ZuseZ4@mejrs@rustbot

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

Skip to content
Fix autodiff_illegal.rs test - #160768
Merged
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom 
JonathanBrouwer:fix_autodiff_ci
Aug 9, 2026
Merged
Fix autodiff_illegal.rs test#160768
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom 
JonathanBrouwer:fix_autodiff_ci
Conversation
@JonathanBrouwer
@JonathanBrouwerJonathanBrouwer
commented
Aug 8, 2026•

edited

Loading

Copy link


Copy Markdown

Member
This fixes the current CI failure: #160761 (comment)
r? @ZuseZ4
  •  I did not use an LLM to create a change in this PR.
  •  I used an LLM to create a change in this PR, and I have explained below how it was used.
@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.
labels
Aug 8, 2026
mejrs
mejrs
reviewed
Aug 8, 2026
View reviewed changes
@mejrsmejrs
left a comment
•

edited by rustbot

Loading

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.

Looks like fallout of #160577
Is it correct to enable proc_macro_hygiene here rather than just change the test output? The test did not previously enable stmt_expr_attributes, for example.
Note also that proc_macro_hygiene is sort of a grab bag of things that macros can't be on and we're likely to move individual items to stable rather than stabilize the entire feature at once. As such if you want this test to reflect changes in what positions macros are allowed in it would be better to not enable the feature.
View changes since this review
@JonathanBrouwerJonathanBrouwer
changed the title
Add proc_macro_hygiene feature gate to autodiff_illegal.rs to fix CIFix autodiff_illegal.rs testAug 8, 2026
@JonathanBrouwer
JonathanBrouwer
commented
Aug 8, 2026

Copy link


Copy Markdown

MemberAuthor
@mejrs Agreed, fixed
@JonathanBrouwer
JonathanBrouwer
commented
Aug 8, 2026

Copy link


Copy Markdown

MemberAuthor
@bors try jobs=optional-x86_64-gnu-autodiff
@rust-bors

This comment has been minimized.

Sign in to view
rust-borsBot
pushed a commit
that referenced
this pull request
Aug 8, 2026
@bors
Auto merge of#160768- JonathanBrouwer:fix_autodiff_ci, r=<try>
dbd7f24
Fix autodiff_illegal.rs test
try-job: optional-x86_64-gnu-autodiff
mejrs
mejrs
approved these changes
Aug 8, 2026
View reviewed changes
@mejrsmejrs
left a comment
•

edited by rustbot

Loading

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.

r=me once green unless you want to wait for the autodiff guys to review, but best to merge this soon because the fail message is annoying me already
View changes since this review
@rust-bors
rust-borsBot
commented
Aug 8, 2026

Copy link


Copy Markdown

Contributor
☀️ Try build successful (CI)

Build commit: dbd7f24 (dbd7f2441d0c37ebd150ecacd40f4bf771c6e63b)

Base parent: 771916f (771916f9028e7fe56d2685f2c4f698de5d7d6a45)
@ZuseZ4
ZuseZ4
commented
Aug 8, 2026

Copy link


Copy Markdown

Member
@bors r=mejrs,ZuseZ4 rollup
@rust-bors
rust-borsBot
commented
Aug 8, 2026

Copy link


Copy Markdown

Contributor
📌 Commit acd2f65 has been approved by mejrs,ZuseZ4
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 8, 2026
jhpratt added a commit
to jhpratt/rust
that referenced
this pull request
Aug 9, 2026
@jhpratt
Rollup merge ofrust-lang#160768- JonathanBrouwer:fix_autodiff_ci, r…
866f202
…=mejrs,ZuseZ4
Fix autodiff_illegal.rs test
This fixes the current CI failure: rust-lang#160761 (comment)
r? @ZuseZ4
@jhprattjhpratt
mentioned this pull request
Aug 9, 2026
 Closed

rust-borsBot
pushed a commit
that referenced
this pull request
Aug 9, 2026
@bors
Auto merge of#160786- jhpratt:rollup-Mfq8vBr, r=jhpratt
74dbe2d
Rollup of 6 pull requests
Successful merges:
- #160529 (Upgrade and deduplicate dependencies)
- #158517 (Initial implementation of named `Fn` trait parameters)
- #158587 (check if len of array const arg matches the expected len of the type when lowering to valtree)
- #160748 (Add regression test for cross-crate assoc const private field leak in rustdoc)
- #160708 (remove old update mechanism)
- #160768 (Fix autodiff_illegal.rs test)
@jhprattjhpratt
mentioned this pull request
Aug 9, 2026
 Merged

rust-borsBot
pushed a commit
that referenced
this pull request
Aug 9, 2026
@bors
Auto merge of#160789- jhpratt:rollup-zFeRr02, r=jhpratt
153ecc4
Rollup of 5 pull requests
Successful merges:
- #158517 (Initial implementation of named `Fn` trait parameters)
- #158587 (check if len of array const arg matches the expected len of the type when lowering to valtree)
- #160748 (Add regression test for cross-crate assoc const private field leak in rustdoc)
- #160708 (remove old update mechanism)
- #160768 (Fix autodiff_illegal.rs test)
@rust-bors
rust-borsBot
merged commit f5c9a72
into
rust-lang:mainAug 9, 2026

14 checks passed

@rustbotrustbot
added this to the 1.99.0 milestone
Aug 9, 2026
rust-timer added a commit
that referenced
this pull request
Aug 9, 2026
@rust-timer
Unrolled build for#160768
b7148e0
Rollup merge of #160768 - JonathanBrouwer:fix_autodiff_ci, r=mejrs,ZuseZ4
Fix autodiff_illegal.rs test
This fixes the current CI failure: #160761 (comment)
r? @ZuseZ4
@cjgillotcjgillot
mentioned this pull request
Aug 9, 2026
 Open

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@mejrsmejrsmejrs approved these changes
@ZuseZ4ZuseZ4Awaiting requested review from ZuseZ4

Assignees

@ZuseZ4ZuseZ4

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.

Projects


None yet


Milestone

1.99.0

Development

Successfully merging this pull request may close these issues.

4 participants

@JonathanBrouwer@ZuseZ4@mejrs@rustbot

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

Skip to content
Fix autodiff_illegal.rs test - #160768
Merged
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom 
JonathanBrouwer:fix_autodiff_ci
Aug 9, 2026
Merged
Fix autodiff_illegal.rs test#160768
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom 
JonathanBrouwer:fix_autodiff_ci
Conversation
@JonathanBrouwer
@JonathanBrouwerJonathanBrouwer
commented
Aug 8, 2026•

edited

Loading

Copy link


Copy Markdown

Member
This fixes the current CI failure: #160761 (comment)
r? @ZuseZ4
  •  I did not use an LLM to create a change in this PR.
  •  I used an LLM to create a change in this PR, and I have explained below how it was used.
@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.
labels
Aug 8, 2026
mejrs
mejrs
reviewed
Aug 8, 2026
View reviewed changes
@mejrsmejrs
left a comment
•

edited by rustbot

Loading

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.

Looks like fallout of #160577
Is it correct to enable proc_macro_hygiene here rather than just change the test output? The test did not previously enable stmt_expr_attributes, for example.
Note also that proc_macro_hygiene is sort of a grab bag of things that macros can't be on and we're likely to move individual items to stable rather than stabilize the entire feature at once. As such if you want this test to reflect changes in what positions macros are allowed in it would be better to not enable the feature.
View changes since this review
@JonathanBrouwerJonathanBrouwer
changed the title
Add proc_macro_hygiene feature gate to autodiff_illegal.rs to fix CIFix autodiff_illegal.rs testAug 8, 2026
@JonathanBrouwer
JonathanBrouwer
commented
Aug 8, 2026

Copy link


Copy Markdown

MemberAuthor
@mejrs Agreed, fixed
@JonathanBrouwer
JonathanBrouwer
commented
Aug 8, 2026

Copy link


Copy Markdown

MemberAuthor
@bors try jobs=optional-x86_64-gnu-autodiff
@rust-bors

This comment has been minimized.

Sign in to view
rust-borsBot
pushed a commit
that referenced
this pull request
Aug 8, 2026
@bors
Auto merge of#160768- JonathanBrouwer:fix_autodiff_ci, r=<try>
dbd7f24
Fix autodiff_illegal.rs test
try-job: optional-x86_64-gnu-autodiff
mejrs
mejrs
approved these changes
Aug 8, 2026
View reviewed changes
@mejrsmejrs
left a comment
•

edited by rustbot

Loading

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.

r=me once green unless you want to wait for the autodiff guys to review, but best to merge this soon because the fail message is annoying me already
View changes since this review
@rust-bors
rust-borsBot
commented
Aug 8, 2026

Copy link


Copy Markdown

Contributor
☀️ Try build successful (CI)

Build commit: dbd7f24 (dbd7f2441d0c37ebd150ecacd40f4bf771c6e63b)

Base parent: 771916f (771916f9028e7fe56d2685f2c4f698de5d7d6a45)
@ZuseZ4
ZuseZ4
commented
Aug 8, 2026

Copy link


Copy Markdown

Member
@bors r=mejrs,ZuseZ4 rollup
@rust-bors
rust-borsBot
commented
Aug 8, 2026

Copy link


Copy Markdown

Contributor
📌 Commit acd2f65 has been approved by mejrs,ZuseZ4
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 8, 2026
jhpratt added a commit
to jhpratt/rust
that referenced
this pull request
Aug 9, 2026
@jhpratt
Rollup merge ofrust-lang#160768- JonathanBrouwer:fix_autodiff_ci, r…
866f202
…=mejrs,ZuseZ4
Fix autodiff_illegal.rs test
This fixes the current CI failure: rust-lang#160761 (comment)
r? @ZuseZ4
@jhprattjhpratt
mentioned this pull request
Aug 9, 2026
 Closed

rust-borsBot
pushed a commit
that referenced
this pull request
Aug 9, 2026
@bors
Auto merge of#160786- jhpratt:rollup-Mfq8vBr, r=jhpratt
74dbe2d
Rollup of 6 pull requests
Successful merges:
- #160529 (Upgrade and deduplicate dependencies)
- #158517 (Initial implementation of named `Fn` trait parameters)
- #158587 (check if len of array const arg matches the expected len of the type when lowering to valtree)
- #160748 (Add regression test for cross-crate assoc const private field leak in rustdoc)
- #160708 (remove old update mechanism)
- #160768 (Fix autodiff_illegal.rs test)
@jhprattjhpratt
mentioned this pull request
Aug 9, 2026
 Merged

rust-borsBot
pushed a commit
that referenced
this pull request
Aug 9, 2026
@bors
Auto merge of#160789- jhpratt:rollup-zFeRr02, r=jhpratt
153ecc4
Rollup of 5 pull requests
Successful merges:
- #158517 (Initial implementation of named `Fn` trait parameters)
- #158587 (check if len of array const arg matches the expected len of the type when lowering to valtree)
- #160748 (Add regression test for cross-crate assoc const private field leak in rustdoc)
- #160708 (remove old update mechanism)
- #160768 (Fix autodiff_illegal.rs test)
@rust-bors
rust-borsBot
merged commit f5c9a72
into
rust-lang:mainAug 9, 2026

14 checks passed

@rustbotrustbot
added this to the 1.99.0 milestone
Aug 9, 2026
rust-timer added a commit
that referenced
this pull request
Aug 9, 2026
@rust-timer
Unrolled build for#160768
b7148e0
Rollup merge of #160768 - JonathanBrouwer:fix_autodiff_ci, r=mejrs,ZuseZ4
Fix autodiff_illegal.rs test
This fixes the current CI failure: #160761 (comment)
r? @ZuseZ4
@cjgillotcjgillot
mentioned this pull request
Aug 9, 2026
 Open

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@mejrsmejrsmejrs approved these changes
@ZuseZ4ZuseZ4Awaiting requested review from ZuseZ4

Assignees

@ZuseZ4ZuseZ4

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.

Projects


None yet


Milestone

1.99.0

Development

Successfully merging this pull request may close these issues.

4 participants

@JonathanBrouwer@ZuseZ4@mejrs@rustbot

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Strip utm_, fbclid, gclid, etc. from all links on page
(function() {
var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',
'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',
'ref', 'ref_src', 'source', 'medium', 'campaign'];
function cleanUrl(url) {
try {
var u = new URL(url, window.location.origin);
var changed = false;
trackingParams.forEach(function(p) {
if (u.searchParams.has(p)) {
u.searchParams.delete(p);
changed = true;
}
});
return changed ? u.toString() : url;
} catch (e) {
return url;
}
}
function cleanLinks() {
document.querySelectorAll('a[href]').forEach(function(a) {
var clean = cleanUrl(a.href);
if (clean !== a.href) a.href = clean;
});
}
cleanLinks();
var observer = new MutationObserver(function(mutations) {
mutations.forEach(function(m) {
m.addedNodes.forEach(function(node) {
if (node.nodeType === 1) {
if (node.tagName === 'A') cleanLinks();
node.querySelectorAll('a[href]').forEach(function(a) {
var clean = cleanUrl(a.href);
if (clean !== a.href) a.href = clean;
});
}
});
});
});
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} 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
Fix autodiff_illegal.rs test - #160768
Merged
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom 
JonathanBrouwer:fix_autodiff_ci
Aug 9, 2026
Merged
Fix autodiff_illegal.rs test#160768
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom 
JonathanBrouwer:fix_autodiff_ci
Conversation
@JonathanBrouwer
@JonathanBrouwerJonathanBrouwer
commented
Aug 8, 2026•

edited

Loading

Copy link


Copy Markdown

Member
This fixes the current CI failure: #160761 (comment)
r? @ZuseZ4
  •  I did not use an LLM to create a change in this PR.
  •  I used an LLM to create a change in this PR, and I have explained below how it was used.
@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.
labels
Aug 8, 2026
mejrs
mejrs
reviewed
Aug 8, 2026
View reviewed changes
@mejrsmejrs
left a comment
•

edited by rustbot

Loading

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.

Looks like fallout of #160577
Is it correct to enable proc_macro_hygiene here rather than just change the test output? The test did not previously enable stmt_expr_attributes, for example.
Note also that proc_macro_hygiene is sort of a grab bag of things that macros can't be on and we're likely to move individual items to stable rather than stabilize the entire feature at once. As such if you want this test to reflect changes in what positions macros are allowed in it would be better to not enable the feature.
View changes since this review
@JonathanBrouwerJonathanBrouwer
changed the title
Add proc_macro_hygiene feature gate to autodiff_illegal.rs to fix CIFix autodiff_illegal.rs testAug 8, 2026
@JonathanBrouwer
JonathanBrouwer
commented
Aug 8, 2026

Copy link


Copy Markdown

MemberAuthor
@mejrs Agreed, fixed
@JonathanBrouwer
JonathanBrouwer
commented
Aug 8, 2026

Copy link


Copy Markdown

MemberAuthor
@bors try jobs=optional-x86_64-gnu-autodiff
@rust-bors

This comment has been minimized.

Sign in to view
rust-borsBot
pushed a commit
that referenced
this pull request
Aug 8, 2026
@bors
Auto merge of#160768- JonathanBrouwer:fix_autodiff_ci, r=<try>
dbd7f24
Fix autodiff_illegal.rs test
try-job: optional-x86_64-gnu-autodiff
mejrs
mejrs
approved these changes
Aug 8, 2026
View reviewed changes
@mejrsmejrs
left a comment
•

edited by rustbot

Loading

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.

r=me once green unless you want to wait for the autodiff guys to review, but best to merge this soon because the fail message is annoying me already
View changes since this review
@rust-bors
rust-borsBot
commented
Aug 8, 2026

Copy link


Copy Markdown

Contributor
☀️ Try build successful (CI)

Build commit: dbd7f24 (dbd7f2441d0c37ebd150ecacd40f4bf771c6e63b)

Base parent: 771916f (771916f9028e7fe56d2685f2c4f698de5d7d6a45)
@ZuseZ4
ZuseZ4
commented
Aug 8, 2026

Copy link


Copy Markdown

Member
@bors r=mejrs,ZuseZ4 rollup
@rust-bors
rust-borsBot
commented
Aug 8, 2026

Copy link


Copy Markdown

Contributor
📌 Commit acd2f65 has been approved by mejrs,ZuseZ4
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 8, 2026
jhpratt added a commit
to jhpratt/rust
that referenced
this pull request
Aug 9, 2026
@jhpratt
Rollup merge ofrust-lang#160768- JonathanBrouwer:fix_autodiff_ci, r…
866f202
…=mejrs,ZuseZ4
Fix autodiff_illegal.rs test
This fixes the current CI failure: rust-lang#160761 (comment)
r? @ZuseZ4
@jhprattjhpratt
mentioned this pull request
Aug 9, 2026
 Closed

rust-borsBot
pushed a commit
that referenced
this pull request
Aug 9, 2026
@bors
Auto merge of#160786- jhpratt:rollup-Mfq8vBr, r=jhpratt
74dbe2d
Rollup of 6 pull requests
Successful merges:
- #160529 (Upgrade and deduplicate dependencies)
- #158517 (Initial implementation of named `Fn` trait parameters)
- #158587 (check if len of array const arg matches the expected len of the type when lowering to valtree)
- #160748 (Add regression test for cross-crate assoc const private field leak in rustdoc)
- #160708 (remove old update mechanism)
- #160768 (Fix autodiff_illegal.rs test)
@jhprattjhpratt
mentioned this pull request
Aug 9, 2026
 Merged

rust-borsBot
pushed a commit
that referenced
this pull request
Aug 9, 2026
@bors
Auto merge of#160789- jhpratt:rollup-zFeRr02, r=jhpratt
153ecc4
Rollup of 5 pull requests
Successful merges:
- #158517 (Initial implementation of named `Fn` trait parameters)
- #158587 (check if len of array const arg matches the expected len of the type when lowering to valtree)
- #160748 (Add regression test for cross-crate assoc const private field leak in rustdoc)
- #160708 (remove old update mechanism)
- #160768 (Fix autodiff_illegal.rs test)
@rust-bors
rust-borsBot
merged commit f5c9a72
into
rust-lang:mainAug 9, 2026

14 checks passed

@rustbotrustbot
added this to the 1.99.0 milestone
Aug 9, 2026
rust-timer added a commit
that referenced
this pull request
Aug 9, 2026
@rust-timer
Unrolled build for#160768
b7148e0
Rollup merge of #160768 - JonathanBrouwer:fix_autodiff_ci, r=mejrs,ZuseZ4
Fix autodiff_illegal.rs test
This fixes the current CI failure: #160761 (comment)
r? @ZuseZ4
@cjgillotcjgillot
mentioned this pull request
Aug 9, 2026
 Open

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@mejrsmejrsmejrs approved these changes
@ZuseZ4ZuseZ4Awaiting requested review from ZuseZ4

Assignees

@ZuseZ4ZuseZ4

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.

Projects


None yet


Milestone

1.99.0

Development

Successfully merging this pull request may close these issues.

4 participants

@JonathanBrouwer@ZuseZ4@mejrs@rustbot

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

Skip to content
Fix autodiff_illegal.rs test - #160768
Merged
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom 
JonathanBrouwer:fix_autodiff_ci
Aug 9, 2026
Merged
Fix autodiff_illegal.rs test#160768
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom 
JonathanBrouwer:fix_autodiff_ci
Conversation
@JonathanBrouwer
@JonathanBrouwerJonathanBrouwer
commented
Aug 8, 2026•

edited

Loading

Copy link


Copy Markdown

Member
This fixes the current CI failure: #160761 (comment)
r? @ZuseZ4
  •  I did not use an LLM to create a change in this PR.
  •  I used an LLM to create a change in this PR, and I have explained below how it was used.
@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.
labels
Aug 8, 2026
mejrs
mejrs
reviewed
Aug 8, 2026
View reviewed changes
@mejrsmejrs
left a comment
•

edited by rustbot

Loading

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.

Looks like fallout of #160577
Is it correct to enable proc_macro_hygiene here rather than just change the test output? The test did not previously enable stmt_expr_attributes, for example.
Note also that proc_macro_hygiene is sort of a grab bag of things that macros can't be on and we're likely to move individual items to stable rather than stabilize the entire feature at once. As such if you want this test to reflect changes in what positions macros are allowed in it would be better to not enable the feature.
View changes since this review
@JonathanBrouwerJonathanBrouwer
changed the title
Add proc_macro_hygiene feature gate to autodiff_illegal.rs to fix CIFix autodiff_illegal.rs testAug 8, 2026
@JonathanBrouwer
JonathanBrouwer
commented
Aug 8, 2026

Copy link


Copy Markdown

MemberAuthor
@mejrs Agreed, fixed
@JonathanBrouwer
JonathanBrouwer
commented
Aug 8, 2026

Copy link


Copy Markdown

MemberAuthor
@bors try jobs=optional-x86_64-gnu-autodiff
@rust-bors

This comment has been minimized.

Sign in to view
rust-borsBot
pushed a commit
that referenced
this pull request
Aug 8, 2026
@bors
Auto merge of#160768- JonathanBrouwer:fix_autodiff_ci, r=<try>
dbd7f24
Fix autodiff_illegal.rs test
try-job: optional-x86_64-gnu-autodiff
mejrs
mejrs
approved these changes
Aug 8, 2026
View reviewed changes
@mejrsmejrs
left a comment
•

edited by rustbot

Loading

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.

r=me once green unless you want to wait for the autodiff guys to review, but best to merge this soon because the fail message is annoying me already
View changes since this review
@rust-bors
rust-borsBot
commented
Aug 8, 2026

Copy link


Copy Markdown

Contributor
☀️ Try build successful (CI)

Build commit: dbd7f24 (dbd7f2441d0c37ebd150ecacd40f4bf771c6e63b)

Base parent: 771916f (771916f9028e7fe56d2685f2c4f698de5d7d6a45)
@ZuseZ4
ZuseZ4
commented
Aug 8, 2026

Copy link


Copy Markdown

Member
@bors r=mejrs,ZuseZ4 rollup
@rust-bors
rust-borsBot
commented
Aug 8, 2026

Copy link


Copy Markdown

Contributor
📌 Commit acd2f65 has been approved by mejrs,ZuseZ4
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 8, 2026
jhpratt added a commit
to jhpratt/rust
that referenced
this pull request
Aug 9, 2026
@jhpratt
Rollup merge ofrust-lang#160768- JonathanBrouwer:fix_autodiff_ci, r…
866f202
…=mejrs,ZuseZ4
Fix autodiff_illegal.rs test
This fixes the current CI failure: rust-lang#160761 (comment)
r? @ZuseZ4
@jhprattjhpratt
mentioned this pull request
Aug 9, 2026
 Closed

rust-borsBot
pushed a commit
that referenced
this pull request
Aug 9, 2026
@bors
Auto merge of#160786- jhpratt:rollup-Mfq8vBr, r=jhpratt
74dbe2d
Rollup of 6 pull requests
Successful merges:
- #160529 (Upgrade and deduplicate dependencies)
- #158517 (Initial implementation of named `Fn` trait parameters)
- #158587 (check if len of array const arg matches the expected len of the type when lowering to valtree)
- #160748 (Add regression test for cross-crate assoc const private field leak in rustdoc)
- #160708 (remove old update mechanism)
- #160768 (Fix autodiff_illegal.rs test)
@jhprattjhpratt
mentioned this pull request
Aug 9, 2026
 Merged

rust-borsBot
pushed a commit
that referenced
this pull request
Aug 9, 2026
@bors
Auto merge of#160789- jhpratt:rollup-zFeRr02, r=jhpratt
153ecc4
Rollup of 5 pull requests
Successful merges:
- #158517 (Initial implementation of named `Fn` trait parameters)
- #158587 (check if len of array const arg matches the expected len of the type when lowering to valtree)
- #160748 (Add regression test for cross-crate assoc const private field leak in rustdoc)
- #160708 (remove old update mechanism)
- #160768 (Fix autodiff_illegal.rs test)
@rust-bors
rust-borsBot
merged commit f5c9a72
into
rust-lang:mainAug 9, 2026

14 checks passed

@rustbotrustbot
added this to the 1.99.0 milestone
Aug 9, 2026
rust-timer added a commit
that referenced
this pull request
Aug 9, 2026
@rust-timer
Unrolled build for#160768
b7148e0
Rollup merge of #160768 - JonathanBrouwer:fix_autodiff_ci, r=mejrs,ZuseZ4
Fix autodiff_illegal.rs test
This fixes the current CI failure: #160761 (comment)
r? @ZuseZ4
@cjgillotcjgillot
mentioned this pull request
Aug 9, 2026
 Open

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@mejrsmejrsmejrs approved these changes
@ZuseZ4ZuseZ4Awaiting requested review from ZuseZ4

Assignees

@ZuseZ4ZuseZ4

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.

Projects


None yet


Milestone

1.99.0

Development

Successfully merging this pull request may close these issues.

4 participants

@JonathanBrouwer@ZuseZ4@mejrs@rustbot

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

Skip to content
Fix autodiff_illegal.rs test - #160768
Merged
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom 
JonathanBrouwer:fix_autodiff_ci
Aug 9, 2026
Merged
Fix autodiff_illegal.rs test#160768
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom 
JonathanBrouwer:fix_autodiff_ci
Conversation
@JonathanBrouwer
@JonathanBrouwerJonathanBrouwer
commented
Aug 8, 2026•

edited

Loading

Copy link


Copy Markdown

Member
This fixes the current CI failure: #160761 (comment)
r? @ZuseZ4
  •  I did not use an LLM to create a change in this PR.
  •  I used an LLM to create a change in this PR, and I have explained below how it was used.
@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.
labels
Aug 8, 2026
mejrs
mejrs
reviewed
Aug 8, 2026
View reviewed changes
@mejrsmejrs
left a comment
•

edited by rustbot

Loading

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.

Looks like fallout of #160577
Is it correct to enable proc_macro_hygiene here rather than just change the test output? The test did not previously enable stmt_expr_attributes, for example.
Note also that proc_macro_hygiene is sort of a grab bag of things that macros can't be on and we're likely to move individual items to stable rather than stabilize the entire feature at once. As such if you want this test to reflect changes in what positions macros are allowed in it would be better to not enable the feature.
View changes since this review
@JonathanBrouwerJonathanBrouwer
changed the title
Add proc_macro_hygiene feature gate to autodiff_illegal.rs to fix CIFix autodiff_illegal.rs testAug 8, 2026
@JonathanBrouwer
JonathanBrouwer
commented
Aug 8, 2026

Copy link


Copy Markdown

MemberAuthor
@mejrs Agreed, fixed
@JonathanBrouwer
JonathanBrouwer
commented
Aug 8, 2026

Copy link


Copy Markdown

MemberAuthor
@bors try jobs=optional-x86_64-gnu-autodiff
@rust-bors

This comment has been minimized.

Sign in to view
rust-borsBot
pushed a commit
that referenced
this pull request
Aug 8, 2026
@bors
Auto merge of#160768- JonathanBrouwer:fix_autodiff_ci, r=<try>
dbd7f24
Fix autodiff_illegal.rs test
try-job: optional-x86_64-gnu-autodiff
mejrs
mejrs
approved these changes
Aug 8, 2026
View reviewed changes
@mejrsmejrs
left a comment
•

edited by rustbot

Loading

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.

r=me once green unless you want to wait for the autodiff guys to review, but best to merge this soon because the fail message is annoying me already
View changes since this review
@rust-bors
rust-borsBot
commented
Aug 8, 2026

Copy link


Copy Markdown

Contributor
☀️ Try build successful (CI)

Build commit: dbd7f24 (dbd7f2441d0c37ebd150ecacd40f4bf771c6e63b)

Base parent: 771916f (771916f9028e7fe56d2685f2c4f698de5d7d6a45)
@ZuseZ4
ZuseZ4
commented
Aug 8, 2026

Copy link


Copy Markdown

Member
@bors r=mejrs,ZuseZ4 rollup
@rust-bors
rust-borsBot
commented
Aug 8, 2026

Copy link


Copy Markdown

Contributor
📌 Commit acd2f65 has been approved by mejrs,ZuseZ4
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 8, 2026
jhpratt added a commit
to jhpratt/rust
that referenced
this pull request
Aug 9, 2026
@jhpratt
Rollup merge ofrust-lang#160768- JonathanBrouwer:fix_autodiff_ci, r…
866f202
…=mejrs,ZuseZ4
Fix autodiff_illegal.rs test
This fixes the current CI failure: rust-lang#160761 (comment)
r? @ZuseZ4
@jhprattjhpratt
mentioned this pull request
Aug 9, 2026
 Closed

rust-borsBot
pushed a commit
that referenced
this pull request
Aug 9, 2026
@bors
Auto merge of#160786- jhpratt:rollup-Mfq8vBr, r=jhpratt
74dbe2d
Rollup of 6 pull requests
Successful merges:
- #160529 (Upgrade and deduplicate dependencies)
- #158517 (Initial implementation of named `Fn` trait parameters)
- #158587 (check if len of array const arg matches the expected len of the type when lowering to valtree)
- #160748 (Add regression test for cross-crate assoc const private field leak in rustdoc)
- #160708 (remove old update mechanism)
- #160768 (Fix autodiff_illegal.rs test)
@jhprattjhpratt
mentioned this pull request
Aug 9, 2026
 Merged

rust-borsBot
pushed a commit
that referenced
this pull request
Aug 9, 2026
@bors
Auto merge of#160789- jhpratt:rollup-zFeRr02, r=jhpratt
153ecc4
Rollup of 5 pull requests
Successful merges:
- #158517 (Initial implementation of named `Fn` trait parameters)
- #158587 (check if len of array const arg matches the expected len of the type when lowering to valtree)
- #160748 (Add regression test for cross-crate assoc const private field leak in rustdoc)
- #160708 (remove old update mechanism)
- #160768 (Fix autodiff_illegal.rs test)
@rust-bors
rust-borsBot
merged commit f5c9a72
into
rust-lang:mainAug 9, 2026

14 checks passed

@rustbotrustbot
added this to the 1.99.0 milestone
Aug 9, 2026
rust-timer added a commit
that referenced
this pull request
Aug 9, 2026
@rust-timer
Unrolled build for#160768
b7148e0
Rollup merge of #160768 - JonathanBrouwer:fix_autodiff_ci, r=mejrs,ZuseZ4
Fix autodiff_illegal.rs test
This fixes the current CI failure: #160761 (comment)
r? @ZuseZ4
@cjgillotcjgillot
mentioned this pull request
Aug 9, 2026
 Open

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@mejrsmejrsmejrs approved these changes
@ZuseZ4ZuseZ4Awaiting requested review from ZuseZ4

Assignees

@ZuseZ4ZuseZ4

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.

Projects


None yet


Milestone

1.99.0

Development

Successfully merging this pull request may close these issues.

4 participants

@JonathanBrouwer@ZuseZ4@mejrs@rustbot

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

Skip to content
Fix autodiff_illegal.rs test - #160768
Merged
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom 
JonathanBrouwer:fix_autodiff_ci
Aug 9, 2026
Merged
Fix autodiff_illegal.rs test#160768
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom 
JonathanBrouwer:fix_autodiff_ci
Conversation
@JonathanBrouwer
@JonathanBrouwerJonathanBrouwer
commented
Aug 8, 2026•

edited

Loading

Copy link


Copy Markdown

Member
This fixes the current CI failure: #160761 (comment)
r? @ZuseZ4
  •  I did not use an LLM to create a change in this PR.
  •  I used an LLM to create a change in this PR, and I have explained below how it was used.
@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.
labels
Aug 8, 2026
mejrs
mejrs
reviewed
Aug 8, 2026
View reviewed changes
@mejrsmejrs
left a comment
•

edited by rustbot

Loading

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.

Looks like fallout of #160577
Is it correct to enable proc_macro_hygiene here rather than just change the test output? The test did not previously enable stmt_expr_attributes, for example.
Note also that proc_macro_hygiene is sort of a grab bag of things that macros can't be on and we're likely to move individual items to stable rather than stabilize the entire feature at once. As such if you want this test to reflect changes in what positions macros are allowed in it would be better to not enable the feature.
View changes since this review
@JonathanBrouwerJonathanBrouwer
changed the title
Add proc_macro_hygiene feature gate to autodiff_illegal.rs to fix CIFix autodiff_illegal.rs testAug 8, 2026
@JonathanBrouwer
JonathanBrouwer
commented
Aug 8, 2026

Copy link


Copy Markdown

MemberAuthor
@mejrs Agreed, fixed
@JonathanBrouwer
JonathanBrouwer
commented
Aug 8, 2026

Copy link


Copy Markdown

MemberAuthor
@bors try jobs=optional-x86_64-gnu-autodiff
@rust-bors

This comment has been minimized.

Sign in to view
rust-borsBot
pushed a commit
that referenced
this pull request
Aug 8, 2026
@bors
Auto merge of#160768- JonathanBrouwer:fix_autodiff_ci, r=<try>
dbd7f24
Fix autodiff_illegal.rs test
try-job: optional-x86_64-gnu-autodiff
mejrs
mejrs
approved these changes
Aug 8, 2026
View reviewed changes
@mejrsmejrs
left a comment
•

edited by rustbot

Loading

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.

r=me once green unless you want to wait for the autodiff guys to review, but best to merge this soon because the fail message is annoying me already
View changes since this review
@rust-bors
rust-borsBot
commented
Aug 8, 2026

Copy link


Copy Markdown

Contributor
☀️ Try build successful (CI)

Build commit: dbd7f24 (dbd7f2441d0c37ebd150ecacd40f4bf771c6e63b)

Base parent: 771916f (771916f9028e7fe56d2685f2c4f698de5d7d6a45)
@ZuseZ4
ZuseZ4
commented
Aug 8, 2026

Copy link


Copy Markdown

Member
@bors r=mejrs,ZuseZ4 rollup
@rust-bors
rust-borsBot
commented
Aug 8, 2026

Copy link


Copy Markdown

Contributor
📌 Commit acd2f65 has been approved by mejrs,ZuseZ4
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 8, 2026
jhpratt added a commit
to jhpratt/rust
that referenced
this pull request
Aug 9, 2026
@jhpratt
Rollup merge ofrust-lang#160768- JonathanBrouwer:fix_autodiff_ci, r…
866f202
…=mejrs,ZuseZ4
Fix autodiff_illegal.rs test
This fixes the current CI failure: rust-lang#160761 (comment)
r? @ZuseZ4
@jhprattjhpratt
mentioned this pull request
Aug 9, 2026
 Closed

rust-borsBot
pushed a commit
that referenced
this pull request
Aug 9, 2026
@bors
Auto merge of#160786- jhpratt:rollup-Mfq8vBr, r=jhpratt
74dbe2d
Rollup of 6 pull requests
Successful merges:
- #160529 (Upgrade and deduplicate dependencies)
- #158517 (Initial implementation of named `Fn` trait parameters)
- #158587 (check if len of array const arg matches the expected len of the type when lowering to valtree)
- #160748 (Add regression test for cross-crate assoc const private field leak in rustdoc)
- #160708 (remove old update mechanism)
- #160768 (Fix autodiff_illegal.rs test)
@jhprattjhpratt
mentioned this pull request
Aug 9, 2026
 Merged

rust-borsBot
pushed a commit
that referenced
this pull request
Aug 9, 2026
@bors
Auto merge of#160789- jhpratt:rollup-zFeRr02, r=jhpratt
153ecc4
Rollup of 5 pull requests
Successful merges:
- #158517 (Initial implementation of named `Fn` trait parameters)
- #158587 (check if len of array const arg matches the expected len of the type when lowering to valtree)
- #160748 (Add regression test for cross-crate assoc const private field leak in rustdoc)
- #160708 (remove old update mechanism)
- #160768 (Fix autodiff_illegal.rs test)
@rust-bors
rust-borsBot
merged commit f5c9a72
into
rust-lang:mainAug 9, 2026

14 checks passed

@rustbotrustbot
added this to the 1.99.0 milestone
Aug 9, 2026
rust-timer added a commit
that referenced
this pull request
Aug 9, 2026
@rust-timer
Unrolled build for#160768
b7148e0
Rollup merge of #160768 - JonathanBrouwer:fix_autodiff_ci, r=mejrs,ZuseZ4
Fix autodiff_illegal.rs test
This fixes the current CI failure: #160761 (comment)
r? @ZuseZ4
@cjgillotcjgillot
mentioned this pull request
Aug 9, 2026
 Open

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@mejrsmejrsmejrs approved these changes
@ZuseZ4ZuseZ4Awaiting requested review from ZuseZ4

Assignees

@ZuseZ4ZuseZ4

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.

Projects


None yet


Milestone

1.99.0

Development

Successfully merging this pull request may close these issues.

4 participants

@JonathanBrouwer@ZuseZ4@mejrs@rustbot