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

target/riscv: Replace watchpoint value mask comparison value with macro. #955

Merged

Conversation

MarekVCodasip
Copy link
Collaborator

This patch replaces ~(typeof(watchpoint->mask))0 with
WATCHPOINT_IGNORE_DATA_VALUE_MASK. This improves
readability and moves the RISCV target in line with
other targets.

Change-Id: I15ac4d4ee76098b304d9b22f720911ba4329c190
Signed-off-by: Marek Vrbka [email protected]

@MarekVCodasip
Copy link
Collaborator Author

I will rebase it once #954 gets merged

This patch replaces ~(typeof(watchpoint->mask))0 with
WATCHPOINT_IGNORE_DATA_VALUE_MASK. This improves
readability and moves the RISCV target in line with
other targets.

Change-Id: I15ac4d4ee76098b304d9b22f720911ba4329c190
Signed-off-by: Marek Vrbka <[email protected]>
@MarekVCodasip MarekVCodasip force-pushed the use-watchpoint-mask-macro branch from f1cfc4d to 2357237 Compare November 9, 2023 09:11
@MarekVCodasip MarekVCodasip marked this pull request as ready for review November 9, 2023 09:13
@MarekVCodasip
Copy link
Collaborator Author

@aap-sc Mentioned this change in #954. I have requested a review from @timsifive.

@timsifive timsifive merged commit 6f02884 into riscv-collab:riscv Nov 9, 2023
4 checks passed
en-sc pushed a commit to en-sc/riscv-openocd that referenced this pull request Nov 10, 2023
…-mask-macro

target/riscv: Replace watchpoint value mask comparison value with macro.

Change-Id: I6767a2b9709b5dd13f96cc3fdb44052d75ceb585
Signed-off-by: Evgeniy Naydanov <[email protected]>
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