[clang] Set DW_AT_language field to DW_LANG_HIP when AMD clang is used (PR #181738)

via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 20 10:33:12 PST 2026


accauble wrote:

> Is there someone from AMD that can confirm that this is a compatible change with their tooling. I assume tooling (like LLDB) benefitted from the fact that we pretended HIP is C++ (like we do for rust e.g.,). Would this break those use-cases? (maybe look in the `git log` for contributors to HIP and add them to the review here)
> 
> Generally, explaining the motivation in the PR description would be great



> Is there someone from AMD that can confirm that this is a compatible change with their tooling. I assume tooling (like LLDB) benefitted from the fact that we pretended HIP is C++ (like we do for rust e.g.,). Would this break those use-cases? (maybe look in the `git log` for contributors to HIP and add them to the review here)
> 
> Generally, explaining the motivation in the PR description would be great

I have updated the description with some motivation. I can look into providing an update for lldb to assume C++ when it sees DW_LANG_HIP. (A user can also set the language as well). Should that be included in this PR?

https://github.com/llvm/llvm-project/pull/181738


More information about the cfe-commits mailing list