[PATCH] D97702: [clang][modules] Use extensible RTTI for ModuleFileExtension

2021-03-05 Thread Jan Svoboda via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG4295ae96cdf2: [clang][modules] Use extensible RTTI for ModuleFileExtension (authored by jansvoboda11). Changed prior to commit: https://reviews.llvm.org/D97702?vs=327163&id=328453#toc Repository: rG

[PATCH] D97702: [clang][modules] Use extensible RTTI for ModuleFileExtension

2021-03-04 Thread Artem Chikin via Phabricator via cfe-commits
artemcm accepted this revision. artemcm added a comment. This revision is now accepted and ready to land. This looks good to me, thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97702/new/ https://reviews.llvm.org/D97702 _

[PATCH] D97702: [clang][modules] Use extensible RTTI for ModuleFileExtension

2021-03-04 Thread Michael Spencer via Phabricator via cfe-commits
Bigcheese added a comment. Looks fine to me, but it would be good for a Swift person to look at this patch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97702/new/ https://reviews.llvm.org/D97702 ___ cf

[PATCH] D97702: [clang][modules] Use extensible RTTI for ModuleFileExtension

2021-03-01 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 created this revision. jansvoboda11 added reviewers: Bigcheese, dexonsmith. jansvoboda11 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Clang exposes an interface for extending the PCM/PCH file format: `ModuleFileExtension`.