================ @@ -111,6 +111,13 @@ Clang Frontend Potentially Breaking Changes $ clang --target=<your target triple> -print-target-triple <the normalized target triple> +- The ``getTypeAsWritten`` member function has been removed from ``ClassTemplateSpecializationDecl`` and + ``VarTemplateSpecializationDecl``, and a new member function ``getTemplateArgsAsWritten`` returning a + ``const ASTTemplateArgumentListInfo *`` has been added to access to the explicitly written template + argument list of explicit specializations, partial specializations, and explicit instantiations of + class and variable templates. Consequently, the ``hasTypeLoc`` AST matcher will no longer match these + declarations and existing uses should switch to ``TemplateArgumentLoc`` matchers instead. + ---------------- sdkrystian wrote:
> I think we can remove the internal details and go with something shorter, > like: > > ``` > The `hasTypeLoc` AST matcher will no longer match a > `classTemplateSpecializationDecl`; existing uses should switch to > `templateArgumentLoc` or `hasAnyTemplateArgumentLoc` instead. > ``` > > WDYT? @AaronBallman As a strictly AST matcher release note, I like your proposed wording. However, I think it's important to include the removal of `getTypeAsWritten` (and the addition of its replacement, `getTemplateArgsAsWritten`) so tooling authors know how to fix any code which is broken by these changes. Would adding two release notes be preferable? For example: > - ``getTypeAsWritten`` has been removed from > ``ClassTemplateSpecializationDecl`` and ``VarTemplateSpecializationDecl`` in > favor of a new function ``getTemplateArgsAsWritten``. > - The ``hasTypeLoc`` AST matcher will no longer match a > ``classTemplateSpecializationDecl``; existing uses should switch to > ``templateArgumentLoc`` or ``hasAnyTemplateArgumentLoc`` instead. https://github.com/llvm/llvm-project/pull/81642 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits