[PATCH] D46769: [AMDGPU] Change llvm.debugtrap to be a debug breakpoint that can resume execution.

[Tony Tye via Phabricator via llvm-commits]

t-tye added inline comments.


================
Comment at: lib/Target/AMDGPU/SIISelLowering.cpp:4003
+      !Subtarget->isTrapHandlerEnabled())
+    return DAG.getNode(AMDGPUISD::ENDPGM, SL, MVT::Other, Chain);
 
----------------
arsenm wrote:
> This can't be a terminator instruction. This will produce a terminator in the middle of the block verifier error. I think this is already an issue in some case
This was behavior was not introduced by this patch as the original code did this. So can address as a separate patch. Can you point me at examples of how lowering should generate code that introduces new control flow?


Repository:
  rL LLVM

https://reviews.llvm.org/D46769