gpu_performance_api icon indicating copy to clipboard operation
gpu_performance_api copied to clipboard

memset of result buffer only clears first 25%

Open AdamJMiles opened this issue 4 years ago • 2 comments
trafficstars

https://github.com/GPUOpen-Tools/gpu_performance_api/blob/30cd97819afd6f560a2dcd6847f5dde1fba08854/source/examples/dx12/dx12_color_cube/cube_sample.cc#L2311

I'm not sure why the sizeof(int) is there? Did the author think memset took a number of DWORDs to memset instead of bytes?

AdamJMiles avatar Oct 06 '21 15:10 AdamJMiles

Woah! You're right! Thanks for catching that and pointing out the mistake! It looks like the same thing was done in the dx11 example here: https://github.com/GPUOpen-Tools/gpu_performance_api/blob/30cd97819afd6f560a2dcd6847f5dde1fba08854/source/examples/dx11/dx11_triangle/dx11_triangle.cc#L1024

Would you like to submit a pull request with a fix to these two samples?

PLohrmannAMD avatar Oct 06 '21 16:10 PLohrmannAMD

Will do!

https://github.com/GPUOpen-Tools/gpu_performance_api/pull/60

AdamJMiles avatar Oct 06 '21 17:10 AdamJMiles