Skip to content

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