Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Run check-continuations in CI #2720

Merged
merged 4 commits into from
Oct 31, 2023
Merged

Run check-continuations in CI #2720

merged 4 commits into from
Oct 31, 2023

Conversation

Flakebi
Copy link
Member

@Flakebi Flakebi commented Sep 21, 2023

Add check-continuations to the GitHub Actions workflows.

@Flakebi Flakebi requested a review from a team as a code owner September 21, 2023 08:36
@qiaojbao qiaojbao changed the title Run check-continuations in CI [CI] Run check-continuations in CI Sep 22, 2023
@qiaojbao qiaojbao changed the title [CI] Run check-continuations in CI Run check-continuations in CI Sep 25, 2023
@nhaehnle
Copy link
Member

Doesn't actually pass. Part apart from that it LGTM.

@Flakebi
Copy link
Member Author

Flakebi commented Oct 26, 2023

Rebased and now also runs check-continuations-units.

@amdvlk-admin
Copy link

Test summary for commit a22df28

CTS tests (Failed: 0/137949)
  • Built with version 1.3.5.2
  • Ubuntu navi3x, Srdcvk
    • Passed: 35353/68947 (51.3%)
    • Failed: 0/68947 (0.0%)
    • Not Supported: 33594/68947 (48.7%)
    • Warnings: 0/68947 (0.0%)
    Ubuntu navi2x, Srdcvk
    • Passed: 35424/69002 (51.3%)
    • Failed: 0/69002 (0.0%)
    • Not Supported: 33578/69002 (48.7%)
    • Warnings: 0/69002 (0.0%)

nhaehnle
nhaehnle previously approved these changes Oct 26, 2023
The SavedRegisterValues vector was accessed after its allocation went
out of scope, make the value owned to prevent that.
Add check-continuations to the GitHub Actions workflows.
@Flakebi
Copy link
Member Author

Flakebi commented Oct 27, 2023

Sanitizers found the first bug in continuations :)

@amdvlk-admin
Copy link

Test summary for commit 3b34a48

CTS tests (Failed: 0/137941)
  • Built with version 1.3.5.2
  • Ubuntu navi3x, Srdcvk
    • Passed: 35353/68939 (51.3%)
    • Failed: 0/68939 (0.0%)
    • Not Supported: 33586/68939 (48.7%)
    • Warnings: 0/68939 (0.0%)
    Ubuntu navi2x, Srdcvk
    • Passed: 35424/69002 (51.3%)
    • Failed: 0/69002 (0.0%)
    • Not Supported: 33578/69002 (48.7%)
    • Warnings: 0/69002 (0.0%)

nhaehnle
nhaehnle previously approved these changes Oct 27, 2023
Copy link
Member

@nhaehnle nhaehnle left a comment

Choose a reason for hiding this comment

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

Nice :)

Hmm, looks like there's at least one more, actually?

The first loop in the post-process pass should only handle start
functions, not resume functions as their arguments are different.

Fix this by filtering out resume functions.

This fixes a regression introduced in the lgc.rt.shaderstage patch. It
asserted in a registercount test.
@Flakebi
Copy link
Member Author

Flakebi commented Oct 30, 2023

Cherry-pick an internal patch to fix tests.

Copy link
Member

@nhaehnle nhaehnle left a comment

Choose a reason for hiding this comment

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

Thanks!

@Flakebi
Copy link
Member Author

Flakebi commented Oct 31, 2023

retest this please

@amdvlk-admin
Copy link

Test summary for commit 09ef5e5

CTS tests (Failed: 0/138020)
  • Built with version 1.3.5.2
  • Ubuntu navi3x, Srdcvk
    • Passed: 35401/69018 (51.3%)
    • Failed: 0/69018 (0.0%)
    • Not Supported: 33617/69018 (48.7%)
    • Warnings: 0/69018 (0.0%)
    Ubuntu navi2x, Srdcvk
    • Passed: 35424/69002 (51.3%)
    • Failed: 0/69002 (0.0%)
    • Not Supported: 33578/69002 (48.7%)
    • Warnings: 0/69002 (0.0%)

@Flakebi Flakebi merged commit 385111d into GPUOpen-Drivers:dev Oct 31, 2023
@Flakebi Flakebi deleted the cont-ci branch October 31, 2023 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants