Skip to content

Conversation

qxy11
Copy link

@qxy11 qxy11 commented Aug 25, 2025

Add tests for Mock GPU memory space reading where we test the "memory read --space " command and also test using the public API changes.

Summary:
Add tests for Mock GPU memory space reading where we test the "memory read --space <space>" command and also test using the public API changes.
Copy link
Collaborator

@dmpots dmpots left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -0,0 +1,3 @@
C_SOURCES := main.c

include Makefile.rules No newline at end of file
Copy link
Collaborator

Choose a reason for hiding this comment

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

add this missing new line. You can configure VSCode to add new lines at the end of files automatically :)


# GPU Plugins
lldb_enable_mock_gpu_plugin = '@LLDB_ENABLE_MOCK_GPU_PLUGIN@'
if lldb_enable_mock_gpu_plugin == 'ON' or lldb_enable_mock_gpu_plugin == '1':
Copy link
Collaborator

Choose a reason for hiding this comment

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

you should instead use
if lldb_enable_mock_gpu_plugin in ('ON', '1'):

you can change the amdgpu line below as well

@qxy11 qxy11 requested a review from walter-erquinigo August 28, 2025 17:05
@dmpots dmpots merged commit e996227 into clayborg:llvm-server-plugins Oct 14, 2025
5 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.

4 participants