This is an archive of the discontinued LLVM Phabricator instance.

Add support for querying SubstTemplateTypeParm types
AcceptedPublic

Authored by anderslanglands on Nov 11 2022, 1:04 AM.

Details

Summary

Adds CXType_SubstTemplateTypeParm and clang_Type_getReplacementType()

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptNov 11 2022, 1:04 AM
anderslanglands requested review of this revision.Nov 11 2022, 1:04 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 11 2022, 1:04 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript

Thanks for this! You should also add a release note about the new functionality.

clang/include/clang-c/Index.h
2793–2794
clang/tools/libclang/CXType.cpp
129–130
1363–1367
1367

Can you add the newline back?

clang/tools/libclang/libclang.map
170

You should keep this in alphabetical order.

Added release note, incorporated other review notes

aaron.ballman accepted this revision.Nov 18 2022, 11:28 AM

LGTM, thank you!

This revision is now accepted and ready to land.Nov 18 2022, 11:28 AM

Great, thanks! I've got a couple more things to land once this is merged and I've cleaned them up. Do you wanna show me how to do the merge myself?

Arg just noticed an issue: clang_Type_getReplacementType should be in the LLVM_16 block in libclang.map not in the LLVM_13 block. I've blown away my local branch so don't know how to fix the patch. Shall I just make a new patch once this is merged?

Arg just noticed an issue: clang_Type_getReplacementType should be in the LLVM_16 block in libclang.map not in the LLVM_13 block. I've blown away my local branch so don't know how to fix the patch. Shall I just make a new patch once this is merged?

In Phabricator, you can click the "Download Raw Diff" link (right-hand side of the page, near-ish the top) to get the patch back; you should be able to use that to get your local branch back to the same state.

Move getReplacementType to LLVM_16 block