Skip to content

change autodiff tests according to the generated ir#158321

Draft
Sa4dUs wants to merge 1 commit into
rust-lang:mainfrom
Sa4dUs:autodiff-test
Draft

change autodiff tests according to the generated ir#158321
Sa4dUs wants to merge 1 commit into
rust-lang:mainfrom
Sa4dUs:autodiff-test

Conversation

@Sa4dUs

@Sa4dUs Sa4dUs commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

we lost fastcc and some attrs

is this expected or should we do something to fix it?

r? @ZuseZ4

@rustbot rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Jun 23, 2026
@@ -77,7 +77,7 @@ fn f2(f: fn(f32) -> f32, x: f32) -> f32 {
// CHECK-NEXT: Function Attrs
// debug-NEXT: define internal { float, float }
// debug-SAME: (ptr align 4 %x, ptr align 4 %bx_0, ptr align 4 %y, ptr align 4 %by_0)
// release-NEXT: define internal fastcc float
// release-NEXT: define internal fastcc { float, float }

@Sa4dUs Sa4dUs Jun 23, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

this looks weird to me but i've been a bit disconnected from autodiff lately so dunno if it's expected

View changes since the review

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

Labels

S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants