This is an archive of the discontinued LLVM Phabricator instance.

add LLVMGetDINodeTag
ClosedPublic

Authored by davibe on Nov 21 2022, 3:59 AM.

Details

Summary

Hello!

I'm working on a tool that visits debug info it using the llvm-c API. I noticed that there is no way to get the dwarf::Tag of a DINode, so I added LLVMGetDINodeTag

Diff Detail

Event Timeline

davibe created this revision.Nov 21 2022, 3:59 AM
Herald added a project: Restricted Project. · View Herald Transcript
Herald added a subscriber: hiraditya. · View Herald Transcript
davibe requested review of this revision.Nov 21 2022, 3:59 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 21 2022, 3:59 AM
davibe updated this revision to Diff 477080.Nov 21 2022, 11:59 PM

Rebased on the main branch

aprantl accepted this revision.Nov 29 2022, 1:31 PM
This revision is now accepted and ready to land.Nov 29 2022, 1:31 PM

I don't have commit access, could someone please commit this for me?

This comment was removed by davibe.

@aprantl this is the other one

This revision was landed with ongoing or failed builds.Feb 8 2023, 4:29 PM
Closed by commit rGe87cc8a979a8: add LLVMGetDINodeTag to C bindings (authored by davibe, committed by aprantl). · Explain Why
This revision was automatically updated to reflect the committed changes.
chapuni added a subscriber: chapuni.Feb 8 2023, 7:06 PM

Warnings in Release build.

llvm/tools/llvm-c-test/debuginfo.c
211

Could you fold it into assert?

222

ditto.

Looks like this was fixed in

commit 2700d66a054327c2dbb90d8b44b1d5cf3b0043b8
Author: Kazu Hirata <kazu@google.com>
Date:   Wed Feb 8 21:00:45 2023 -0800

    [llvm-c-test] Fix unused variable warnings

Thanks!