Skip to content

[mlgo] Fix test post - #147716 #148259

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

Merged
merged 1 commit into from
Jul 12, 2025
Merged

Conversation

JaydeepChauhan14
Copy link
Contributor

No description provided.

@llvmbot llvmbot added the mlgo label Jul 11, 2025
@llvmbot
Copy link
Member

llvmbot commented Jul 11, 2025

@llvm/pr-subscribers-mlgo

Author: None (JaydeepChauhan14)

Changes

Full diff: https://github.com/llvm/llvm-project/pull/148259.diff

1 Files Affected:

  • (modified) llvm/test/CodeGen/MLRegAlloc/dev-mode-extra-features-logging.ll (+1-1)
diff --git a/llvm/test/CodeGen/MLRegAlloc/dev-mode-extra-features-logging.ll b/llvm/test/CodeGen/MLRegAlloc/dev-mode-extra-features-logging.ll
index 31e120007cb7d..bd8d882cda39b 100644
--- a/llvm/test/CodeGen/MLRegAlloc/dev-mode-extra-features-logging.ll
+++ b/llvm/test/CodeGen/MLRegAlloc/dev-mode-extra-features-logging.ll
@@ -26,7 +26,7 @@
 ; Also, the first eviction problem is significantly less than 300 instructions. Check
 ; that there is a zero value.
 ; Note: we're regex-ing some of the opcodes to avoid test flakyness.
-; CHECK: instructions: 20,{{([0-9]{4})}},1{{([0-9]{3})}},1{{([0-9]{3})}},{{.*}},0,
+; CHECK: instructions: 20,{{([0-9]{4})}},1{{([0-9]{3})}},2{{([0-9]{3})}},{{.*}},0,
 ; Only the candidate virtreg and the 10th LR are included in this problem. Make
 ; sure the other LRs have values of zero. There are 2700 0s followed by some 1s.
 ; There's a limit to how many repetitions can be matched.

@e-kud e-kud requested a review from mtrofin July 11, 2025 16:28
@e-kud
Copy link
Contributor

e-kud commented Jul 11, 2025

@mtrofin we've seen the failure and it most likely because in #147716 1999->2001 in some tests. However we weren't able to validate it.

@boomanaiden154 boomanaiden154 self-requested a review July 11, 2025 16:35
@boomanaiden154
Copy link
Contributor

Give me a little bit of time and I should be able to validate it. mtrofin@ is OOO currently.

Copy link
Contributor

@boomanaiden154 boomanaiden154 left a comment

Choose a reason for hiding this comment

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

This fixes it on my end. Thanks for putting this up!

@e-kud
Copy link
Contributor

e-kud commented Jul 12, 2025

This fixes it on my end. Thanks for putting this up!

Thank you!

@e-kud e-kud merged commit 007fc77 into llvm:main Jul 12, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants