[PATCH] D69632: [libTooling] Add Stencil constructor.
Yitzhak Mandelbaum via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Oct 30 12:19:48 PDT 2019
ymandel created this revision.
ymandel added a reviewer: gribozavr.
Herald added a project: clang.
ymandel added a child revision: D69613: [libTooling] Simplify type structure of `Stencil`s..
Adds a constructor that takes a vector with which to initialize the `Parts`
field and a corresponding free function that forwards to the constructor. These
definitions are needed to assist in transitioning away from `Stencil` as a class
to defining it as a type alias.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D69632
Files:
clang/include/clang/Tooling/Transformer/Stencil.h
Index: clang/include/clang/Tooling/Transformer/Stencil.h
===================================================================
--- clang/include/clang/Tooling/Transformer/Stencil.h
+++ clang/include/clang/Tooling/Transformer/Stencil.h
@@ -90,6 +90,7 @@
class Stencil {
public:
Stencil() = default;
+ Stencil(std::vector<StencilPart> Parts) : Parts(std::move(Parts)) {}
/// Composes a stencil from a series of parts.
template <typename... Ts> static Stencil cat(Ts &&... Parts) {
@@ -140,6 +141,12 @@
template <typename... Ts> Stencil cat(Ts &&... Parts) {
return Stencil::cat(std::forward<Ts>(Parts)...);
}
+/// Convenience wrapper for Stencil constructor of the same type. Declaration
+/// outside of the class supports transition of `Stencil` type to an alias
+/// rather than a class.
+inline Stencil cat(std::vector<StencilPart> Parts) {
+ return Stencil(std::move(Parts));
+}
/// \returns exactly the text provided.
StencilPart text(llvm::StringRef Text);
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D69632.227158.patch
Type: text/x-patch
Size: 983 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20191030/65dad880/attachment-0001.bin>
More information about the cfe-commits
mailing list