[PATCH] D58518: [HIP] change kernel stub name

Artem Belevich via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 21 11:56:10 PST 2019


tra added a comment.

In D58518#1406202 <https://reviews.llvm.org/D58518#1406202>, @t-tye wrote:

> Yes this relates to supporting the debugger.
>
> For the same function being present on both host and device, having the same name is correct as the debugger must set a breakpoint at both places. This is similar to needing to set a breakpoint at every place a function is inlined.


I'm confused. Are you saying that HIP does *not* need a different name for the stub then?


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

https://reviews.llvm.org/D58518





More information about the cfe-commits mailing list