[PATCH] D97670: [AMDGPU] New intrinsic void llvm.amdgcn.s.sethalt(i32)

[Tony Tye via Phabricator via llvm-commits]

t-tye added a comment.

> The expected use case is for frontends to insert this into
> shaders that are to be run under a debugger. The shader can
> then be resumed or single stepped from the point of the call
> under debugger control.

Please check with the AMDGPU debugger team before making these assumptions:-) It turns out that the debugger did recently add support for tracking the wave halt state independent of the breakpoint state so this is fine. However, using this does not notify the debugger. A better alternative is to use the llvm.debug intrinsic that will notify the debugger when executed, and also halt the wave for the debugger to resume.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D97670/new/

https://reviews.llvm.org/D97670