Skip to content

[Mips] Reserve hardware register HWR2 #127775

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 2 commits into from
Feb 20, 2025
Merged

Conversation

yingopq
Copy link
Contributor

@yingopq yingopq commented Feb 19, 2025

Fix pr #127553.
x86_64 failed to run readcyclecounter.ll when enable expensive_check,
it would error "Using an undefined physical register".

@yingopq
Copy link
Contributor Author

yingopq commented Feb 20, 2025

@wzssyqa Could you help review again? I tested this patch on x86_64 which can reproduce this failure when enable expsensive_check, the result was ok.

@wzssyqa
Copy link
Contributor

wzssyqa commented Feb 20, 2025

What's is real do should be in the commit msg subject instead of where we find it.
Such as

MIPS: RegisterInfo Reserved HWR2

Mention what it resolved in the description part of commit msg.

@yingopq
Copy link
Contributor Author

yingopq commented Feb 20, 2025

OK, I would change the commit subject and commit message.

Fix pr llvm#127553.
x86_64 failed to run readcyclecounter.ll when enable expensive_check,
it would error "Using an undefined physical register".
@yingopq yingopq force-pushed the Fix_test_fail_106318 branch from 540f230 to 9e210ca Compare February 20, 2025 03:20
@yingopq yingopq changed the title [Mips] Fix expsensive_check test failure when x86 run readcyclecounte… [Mips] Reserve hardware register HWR2 Feb 20, 2025
@wzssyqa
Copy link
Contributor

wzssyqa commented Feb 20, 2025

Please remove XFAIL in readcyclecounter.ll

@yingopq
Copy link
Contributor Author

yingopq commented Feb 20, 2025

Please remove XFAIL in readcyclecounter.ll

OK, removed.

@wzssyqa wzssyqa merged commit 0c809ea into llvm:main Feb 20, 2025
8 checks passed
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.

2 participants