This is an archive of the discontinued LLVM Phabricator instance.

[clang][ASTImporter] Propagate TemplateArgument::IsDefaulted during import
ClosedPublic

Authored by Michael137 on Jan 27 2023, 6:13 AM.

Details

Summary

With https://reviews.llvm.org/D141826 TemplateArguments have an
additional field that indicates their defaulted-ness. This gets
used during debug-info generation and in the clang::TypePrinter.

This patch copies the field during the import process so consumers
of the ASTImporter can benefit from the other Clang components that
read the field.

Testing

  • Added unit-test
  • Checked that this patch (in addition to a follow-up LLDB patch) fixes current test failures in LLDB

Diff Detail

Event Timeline

Michael137 created this revision.Jan 27 2023, 6:13 AM
Herald added a reviewer: shafik. · View Herald Transcript
Herald added a project: Restricted Project. · View Herald Transcript
Herald added a subscriber: rnkovacs. · View Herald Transcript
Michael137 requested review of this revision.Jan 27 2023, 6:13 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 27 2023, 6:13 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
  • clang-format
aprantl accepted this revision.Jan 27 2023, 8:36 AM
This revision is now accepted and ready to land.Jan 27 2023, 8:36 AM
This revision was landed with ongoing or failed builds.Jan 27 2023, 8:39 AM
This revision was automatically updated to reflect the committed changes.