[llvm] [llvm][Support] Add YAMLSchemeGen for producing YAML Schemes from YAMLTraits (PR #133284)
via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 27 10:29:01 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-llvm-support
Author: None (tgs-sc)
<details>
<summary>Changes</summary>
Introduced a way for generating scheme for validating input YAML. This can be useful to see the full structure of the input YAML file for different llvm based tools that use existing YAML parser, for example clang-format, clang-tidy e.t.c. This commit also can be useful for yaml-language-server.
---
Patch is 34.07 KiB, truncated to 20.00 KiB below, full version: https://github.com/llvm/llvm-project/pull/133284.diff
8 Files Affected:
- (added) llvm/include/llvm/Support/YAMLSchemeGen.h (+497)
- (modified) llvm/include/llvm/Support/YAMLTraits.h (+48-27)
- (modified) llvm/lib/Support/CMakeLists.txt (+1)
- (added) llvm/lib/Support/YAMLSchemeGen.cpp (+163)
- (modified) llvm/lib/Support/YAMLTraits.cpp (+4)
- (modified) llvm/unittests/ObjectYAML/CMakeLists.txt (+1)
- (added) llvm/unittests/ObjectYAML/YAMLSchemeGen.cpp (+103)
- (modified) llvm/utils/gn/secondary/llvm/lib/Support/BUILD.gn (+1)
``````````diff
diff --git a/llvm/include/llvm/Support/YAMLSchemeGen.h b/llvm/include/llvm/Support/YAMLSchemeGen.h
new file mode 100644
index 0000000000000..8100e3f1d175f
--- /dev/null
+++ b/llvm/include/llvm/Support/YAMLSchemeGen.h
@@ -0,0 +1,497 @@
+//===- llvm/Support/YAMLSchemeGen.h -----------------------------*- C++ -*-===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+
+#ifndef LLVM_SUPPORT_YAMLSCHEMEGEN_H
+#define LLVM_SUPPORT_YAMLSCHEMEGEN_H
+
+#include "llvm/Support/YAMLTraits.h"
+
+namespace llvm {
+
+namespace yaml {
+
+class SchemeGen : public IO {
+public:
+ SchemeGen(raw_ostream &RO, void *Ctxt = nullptr, int WrapColumn = 70);
+ ~SchemeGen() override = default;
+
+ IOKind getKind() const override;
+ bool outputting() const override;
+ bool mapTag(StringRef, bool) override;
+ void beginMapping() override;
+ void endMapping() override;
+ bool preflightKey(const char *key, bool, bool, bool &, void *&) override;
+ void postflightKey(void *) override;
+ std::vector<StringRef> keys() override;
+ void beginFlowMapping() override;
+ void endFlowMapping() override;
+ unsigned beginSequence() override;
+ void endSequence() override;
+ bool preflightElement(unsigned, void *&) override;
+ void postflightElement(void *) override;
+ unsigned beginFlowSequence() override;
+ bool preflightFlowElement(unsigned, void *&) override;
+ void postflightFlowElement(void *) override;
+ void endFlowSequence() override;
+ void beginEnumScalar() override;
+ bool matchEnumScalar(const char *, bool) override;
+ bool matchEnumFallback() override;
+ void endEnumScalar() override;
+ bool beginBitSetScalar(bool &) override;
+ bool bitSetMatch(const char *, bool) override;
+ void endBitSetScalar() override;
+ void scalarString(StringRef &, QuotingType) override;
+ void blockScalarString(StringRef &) override;
+ void scalarTag(std::string &) override;
+ NodeKind getNodeKind() override;
+ void setError(const Twine &message) override;
+ bool canElideEmptySequence() override;
+
+ // These are only used by operator<<. They could be private
+ // if that templated operator could be made a friend.
+ void beginDocuments();
+ bool preflightDocument(unsigned);
+ void postflightDocument();
+ void endDocuments();
+
+ // These are needed to yamlize scheme.
+ struct YNode {
+ public:
+ YNode(NodeKind Kind) : Kind(Kind) {}
+
+ NodeKind getNodeKind() const { return Kind; }
+
+ private:
+ NodeKind Kind;
+ };
+
+ class ScalarYNode final : public YNode {
+ StringRef Str;
+ QuotingType QT;
+
+ public:
+ ScalarYNode(StringRef Str, QuotingType QT)
+ : YNode(NodeKind::Scalar), Str(Str), QT(QT) {}
+
+ StringRef getValue() const { return Str; }
+
+ QuotingType getQuotingType() const { return QT; }
+ };
+
+ class MapYNode final : public YNode,
+ SmallVector<std::pair<StringRef, YNode *>, 8> {
+ public:
+ using BaseVector = SmallVector<std::pair<StringRef, YNode *>, 8>;
+
+ MapYNode() : YNode(NodeKind::Map) {}
+
+ using BaseVector::begin;
+ using BaseVector::emplace_back;
+ using BaseVector::end;
+ using BaseVector::size;
+ };
+
+ class SequenceYNode final : public YNode, SmallVector<YNode *, 8> {
+ public:
+ using BaseVector = SmallVector<YNode *, 8>;
+
+ SequenceYNode() : YNode(NodeKind::Sequence) {}
+
+ using BaseVector::operator[];
+ using BaseVector::begin;
+ using BaseVector::emplace_back;
+ using BaseVector::end;
+ using BaseVector::resize;
+ using BaseVector::size;
+ };
+
+private:
+ template <typename YNodeType, typename... YNodeArgs>
+ YNodeType *createYNode(YNodeArgs &&...Args) {
+ auto UPtr = std::make_unique<YNodeType>(std::forward<YNodeArgs>(Args)...);
+ auto *Ptr = UPtr.get();
+ YNodes.emplace_back(std::move(UPtr));
+ return Ptr;
+ }
+
+public:
+ ScalarYNode *createScalarYNode(StringRef Str) {
+ return createYNode<ScalarYNode>(Str, QuotingType::None);
+ }
+
+ MapYNode *createMapYNode() { return createYNode<MapYNode>(); }
+
+ SequenceYNode *createSequenceYNode() { return createYNode<SequenceYNode>(); }
+
+ std::vector<std::unique_ptr<YNode>> YNodes;
+
+ using StringVector = SmallVector<StringRef, 8>;
+
+ class SchemeNode {
+ public:
+ virtual YNode *yamlize(SchemeGen &Gen) const = 0;
+
+ virtual ~SchemeNode() = default;
+ };
+
+ enum class PropertyKind : uint8_t {
+ Properties,
+ Required,
+ Type,
+ Enum,
+ Items,
+ };
+
+ class Scheme;
+
+ class SchemeProperty : public SchemeNode {
+ PropertyKind Kind;
+ StringRef Name;
+
+ public:
+ SchemeProperty(PropertyKind Kind, StringRef Name)
+ : Kind(Kind), Name(Name) {}
+
+ PropertyKind getKind() const { return Kind; }
+
+ StringRef getName() const { return Name; }
+ };
+
+ class PropertiesProperty final
+ : public SchemeProperty,
+ SmallVector<std::pair<StringRef, Scheme *>, 8> {
+ public:
+ using BaseVector = SmallVector<std::pair<StringRef, Scheme *>, 8>;
+
+ PropertiesProperty()
+ : SchemeProperty(PropertyKind::Properties, "properties") {}
+
+ using BaseVector::begin;
+ using BaseVector::emplace_back;
+ using BaseVector::end;
+ using BaseVector::size;
+
+ YNode *yamlize(SchemeGen &Gen) const override {
+ auto *Map = Gen.createMapYNode();
+ for (auto &&[Key, Value] : *this) {
+ auto *Y = Value->yamlize(Gen);
+ Map->emplace_back(Key, Y);
+ }
+ return Map;
+ }
+ };
+
+ class RequiredProperty final : public SchemeProperty, StringVector {
+ public:
+ using BaseVector = StringVector;
+
+ RequiredProperty() : SchemeProperty(PropertyKind::Required, "required") {}
+
+ using BaseVector::begin;
+ using BaseVector::emplace_back;
+ using BaseVector::end;
+ using BaseVector::size;
+
+ YNode *yamlize(SchemeGen &Gen) const override {
+ if (size() == 1) {
+ auto *Scalar = Gen.createScalarYNode(*begin());
+ return Scalar;
+ }
+ auto *Sequence = Gen.createSequenceYNode();
+ for (auto &&Value : *this) {
+ auto *Scalar = Gen.createScalarYNode(Value);
+ Sequence->emplace_back(Scalar);
+ }
+ return Sequence;
+ }
+ };
+
+ class TypeProperty final : public SchemeProperty {
+ StringRef Value = "any";
+
+ public:
+ TypeProperty() : SchemeProperty(PropertyKind::Type, "type") {}
+
+ StringRef getValue() const { return Value; }
+
+ void setValue(StringRef Val) { Value = Val; }
+
+ YNode *yamlize(SchemeGen &Gen) const override {
+ auto *Scalar = Gen.createScalarYNode(Value);
+ return Scalar;
+ }
+ };
+
+ class EnumProperty final : public SchemeProperty, StringVector {
+ public:
+ using BaseVector = StringVector;
+
+ EnumProperty() : SchemeProperty(PropertyKind::Enum, "enum") {}
+
+ using BaseVector::begin;
+ using BaseVector::emplace_back;
+ using BaseVector::end;
+ using BaseVector::size;
+
+ YNode *yamlize(SchemeGen &Gen) const override {
+ auto *Sequence = Gen.createSequenceYNode();
+ for (auto &&Value : *this) {
+ auto *Scalar = Gen.createScalarYNode(Value);
+ Sequence->emplace_back(Scalar);
+ }
+ return Sequence;
+ }
+ };
+
+ class ItemsProperty final : public SchemeProperty, SmallVector<Scheme *, 8> {
+ public:
+ using BaseVector = SmallVector<Scheme *, 8>;
+
+ ItemsProperty() : SchemeProperty(PropertyKind::Items, "items") {}
+
+ using BaseVector::begin;
+ using BaseVector::emplace_back;
+ using BaseVector::end;
+ using BaseVector::size;
+
+ YNode *yamlize(SchemeGen &Gen) const override {
+ auto *Sequence = Gen.createSequenceYNode();
+ for (auto &&Value : *this) {
+ auto *Y = Value->yamlize(Gen);
+ Sequence->emplace_back(Y);
+ }
+ return Sequence;
+ }
+ };
+
+ class Scheme final : public SchemeNode, SmallVector<SchemeProperty *, 8> {
+ public:
+ using BaseVector = SmallVector<SchemeProperty *, 8>;
+
+ Scheme() = default;
+
+ using BaseVector::begin;
+ using BaseVector::emplace_back;
+ using BaseVector::end;
+ using BaseVector::size;
+
+ template <typename PropertyType> PropertyType *findProperty() const {
+ auto P = PropertyType{};
+ auto Found = std::find_if(begin(), end(), [&](auto &&Prop) {
+ return Prop->getKind() == P.getKind();
+ });
+ return Found != end() ? static_cast<PropertyType *>(*Found) : nullptr;
+ }
+
+ template <typename PropertyType> bool hasProperty() const {
+ return findProperty<PropertyType>() != nullptr;
+ }
+
+ YNode *yamlize(SchemeGen &Gen) const override {
+ auto *Map = Gen.createMapYNode();
+ for (auto &&Value : *this) {
+ auto *Y = Value->yamlize(Gen);
+ auto Name = Value->getName();
+ Map->emplace_back(Name, Y);
+ }
+ return Map;
+ }
+ };
+
+private:
+ std::vector<std::unique_ptr<SchemeNode>> SchemeNodes;
+ SmallVector<Scheme *, 8> Schemes;
+
+ template <typename PropertyType> PropertyType *createProperty() {
+ auto UPtr = std::make_unique<PropertyType>();
+ auto *Ptr = UPtr.get();
+ SchemeNodes.emplace_back(std::move(UPtr));
+ return Ptr;
+ }
+
+public:
+ template <typename PropertyType>
+ PropertyType *getOrCreateProperty(Scheme &S) {
+ auto Found = S.findProperty<PropertyType>();
+ if (!Found) {
+ Found = createProperty<PropertyType>();
+ S.emplace_back(Found);
+ }
+ return Found;
+ }
+
+ Scheme *createScheme() {
+ auto UPtr = std::make_unique<Scheme>();
+ auto *Ptr = UPtr.get();
+ SchemeNodes.emplace_back(std::move(UPtr));
+ return Ptr;
+ }
+
+ Scheme *getTopScheme() const {
+ return Schemes.empty() ? nullptr : Schemes.back();
+ }
+
+private:
+ Output O;
+ SchemeNode *Root = nullptr;
+};
+
+template <> struct PolymorphicTraits<SchemeGen::YNode> {
+ static NodeKind getKind(const SchemeGen::YNode &N) { return N.getNodeKind(); }
+
+ static SchemeGen::ScalarYNode &getAsScalar(SchemeGen::YNode &N) {
+ return (SchemeGen::ScalarYNode &)N;
+ }
+
+ static SchemeGen::MapYNode &getAsMap(SchemeGen::YNode &N) {
+ return (SchemeGen::MapYNode &)N;
+ }
+
+ static SchemeGen::SequenceYNode &getAsSequence(SchemeGen::YNode &N) {
+ return (SchemeGen::SequenceYNode &)N;
+ }
+};
+
+template <> struct ScalarTraits<SchemeGen::ScalarYNode> {
+ static void output(const SchemeGen::ScalarYNode &N, void *Ctx,
+ llvm::raw_ostream &Out) {
+ Out << N.getValue();
+ }
+
+ static StringRef input(StringRef Scalar, void *Ctx,
+ SchemeGen::ScalarYNode &N) {
+ return {};
+ }
+
+ static QuotingType mustQuote(StringRef) { return QuotingType::None; }
+};
+
+template <> struct MappingTraits<SchemeGen::MapYNode> {
+ static void mapping(IO &io, SchemeGen::MapYNode &N) {
+ for (auto &&[Key, Value] : N) {
+ io.mapRequired(Key.data(), *Value);
+ }
+ }
+};
+
+template <> struct SequenceTraits<SchemeGen::SequenceYNode> {
+ static size_t size(IO &io, SchemeGen::SequenceYNode &N) { return N.size(); }
+
+ static SchemeGen::YNode &element(IO &, SchemeGen::SequenceYNode &N,
+ size_t Idx) {
+ if (Idx >= N.size())
+ N.resize(Idx + 1);
+ return *(N[Idx]);
+ }
+};
+
+// Define non-member operator<< so that Output can stream out document list.
+template <typename T>
+inline std::enable_if_t<has_DocumentListTraits<T>::value, SchemeGen &>
+operator<<(SchemeGen &Gen, T &DocList) {
+ EmptyContext Ctx;
+ Gen.beginDocuments();
+ const size_t count = DocumentListTraits<T>::size(Gen, DocList);
+ for (size_t i = 0; i < count; ++i) {
+ if (Gen.preflightDocument(i)) {
+ yamlize(Gen, DocumentListTraits<T>::element(Gen, DocList, i), true, Ctx);
+ Gen.postflightDocument();
+ }
+ }
+ Gen.endDocuments();
+ return Gen;
+}
+
+// Define non-member operator<< so that Output can stream out a map.
+template <typename T>
+inline std::enable_if_t<has_MappingTraits<T, EmptyContext>::value, SchemeGen &>
+operator<<(SchemeGen &Gen, T &Map) {
+ EmptyContext Ctx;
+ Gen.beginDocuments();
+ if (Gen.preflightDocument(0)) {
+ yamlize(Gen, Map, true, Ctx);
+ Gen.postflightDocument();
+ }
+ Gen.endDocuments();
+ return Gen;
+}
+
+// Define non-member operator<< so that Output can stream out a sequence.
+template <typename T>
+inline std::enable_if_t<has_SequenceTraits<T>::value, SchemeGen &>
+operator<<(SchemeGen &Gen, T &Seq) {
+ EmptyContext Ctx;
+ Gen.beginDocuments();
+ if (Gen.preflightDocument(0)) {
+ yamlize(Gen, Seq, true, Ctx);
+ Gen.postflightDocument();
+ }
+ Gen.endDocuments();
+ return Gen;
+}
+
+// Define non-member operator<< so that Output can stream out a block scalar.
+template <typename T>
+inline std::enable_if_t<has_BlockScalarTraits<T>::value, SchemeGen &>
+operator<<(SchemeGen &Gen, T &Val) {
+ EmptyContext Ctx;
+ Gen.beginDocuments();
+ if (Gen.preflightDocument(0)) {
+ yamlize(Gen, Val, true, Ctx);
+ Gen.postflightDocument();
+ }
+ Gen.endDocuments();
+ return Gen;
+}
+
+// Define non-member operator<< so that Output can stream out a string map.
+template <typename T>
+inline std::enable_if_t<has_CustomMappingTraits<T>::value, SchemeGen &>
+operator<<(SchemeGen &Gen, T &Val) {
+ EmptyContext Ctx;
+ Gen.beginDocuments();
+ if (Gen.preflightDocument(0)) {
+ yamlize(Gen, Val, true, Ctx);
+ Gen.postflightDocument();
+ }
+ Gen.endDocuments();
+ return Gen;
+}
+
+// Define non-member operator<< so that Output can stream out a polymorphic
+// type.
+template <typename T>
+inline std::enable_if_t<has_PolymorphicTraits<T>::value, SchemeGen &>
+operator<<(SchemeGen &Gen, T &Val) {
+ EmptyContext Ctx;
+ Gen.beginDocuments();
+ if (Gen.preflightDocument(0)) {
+ // FIXME: The parser does not support explicit documents terminated with a
+ // plain scalar; the end-marker is included as part of the scalar token.
+ assert(PolymorphicTraits<T>::getKind(Val) != NodeKind::Scalar &&
+ "plain scalar documents are not supported");
+ yamlize(Gen, Val, true, Ctx);
+ Gen.postflightDocument();
+ }
+ Gen.endDocuments();
+ return Gen;
+}
+
+// Provide better error message about types missing a trait specialization
+template <typename T>
+inline std::enable_if_t<missingTraits<T, EmptyContext>::value, SchemeGen &>
+operator<<(SchemeGen &Gen, T &seq) {
+ char missing_yaml_trait_for_type[sizeof(MissingTrait<T>)];
+ return Gen;
+}
+
+} // namespace yaml
+
+} // namespace llvm
+
+#endif // LLVM_SUPPORT_YAMLSCHEMEGEN_H
diff --git a/llvm/include/llvm/Support/YAMLTraits.h b/llvm/include/llvm/Support/YAMLTraits.h
index e707a445012b5..61f90fbdc5ac8 100644
--- a/llvm/include/llvm/Support/YAMLTraits.h
+++ b/llvm/include/llvm/Support/YAMLTraits.h
@@ -774,12 +774,19 @@ struct unvalidatedMappingTraits
bool, has_MappingTraits<T, Context>::value &&
!has_MappingValidateTraits<T, Context>::value> {};
+enum class IOKind : uint8_t {
+ Outputting,
+ Inputting,
+ SchemeGenering,
+};
+
// Base class for Input and Output.
class IO {
public:
IO(void *Ctxt = nullptr);
virtual ~IO();
+ virtual IOKind getKind() const = 0;
virtual bool outputting() const = 0;
virtual unsigned beginSequence() = 0;
@@ -824,15 +831,17 @@ class IO {
template <typename T>
void enumCase(T &Val, const char* Str, const T ConstVal) {
- if ( matchEnumScalar(Str, outputting() && Val == ConstVal) ) {
+ if (matchEnumScalar(Str,
+ getKind() == IOKind::Outputting && Val == ConstVal)) {
Val = ConstVal;
}
}
// allow anonymous enum values to be used with LLVM_YAML_STRONG_TYPEDEF
template <typename T>
- void enumCase(T &Val, const char* Str, const uint32_t ConstVal) {
- if ( matchEnumScalar(Str, outputting() && Val == static_cast<T>(ConstVal)) ) {
+ void enumCase(T &Val, const char *Str, const uint32_t ConstVal) {
+ if (matchEnumScalar(Str, getKind() == IOKind::Outputting &&
+ Val == static_cast<T>(ConstVal))) {
Val = ConstVal;
}
}
@@ -849,8 +858,9 @@ class IO {
}
template <typename T>
- void bitSetCase(T &Val, const char* Str, const T ConstVal) {
- if ( bitSetMatch(Str, outputting() && (Val & ConstVal) == ConstVal) ) {
+ void bitSetCase(T &Val, const char *Str, const T ConstVal) {
+ if (bitSetMatch(Str, getKind() == IOKind::Outputting &&
+ (Val & ConstVal) == ConstVal)) {
Val = static_cast<T>(Val | ConstVal);
}
}
@@ -858,21 +868,24 @@ class IO {
// allow anonymous enum values to be used with LLVM_YAML_STRONG_TYPEDEF
template <typename T>
void bitSetCase(T &Val, const char* Str, const uint32_t ConstVal) {
- if ( bitSetMatch(Str, outputting() && (Val & ConstVal) == ConstVal) ) {
+ if (bitSetMatch(Str, getKind() == IOKind::Outputting &&
+ (Val & ConstVal) == ConstVal)) {
Val = static_cast<T>(Val | ConstVal);
}
}
template <typename T>
void maskedBitSetCase(T &Val, const char *Str, T ConstVal, T Mask) {
- if (bitSetMatch(Str, outputting() && (Val & Mask) == ConstVal))
+ if (bitSetMatch(Str, getKind() == IOKind::Outputting &&
+ (Val & Mask) == ConstVal))
Val = Val | ConstVal;
}
template <typename T>
void maskedBitSetCase(T &Val, const char *Str, uint32_t ConstVal,
uint32_t Mask) {
- if (bitSetMatch(Str, outputting() && (Val & Mask) == ConstVal))
+ if (bitSetMatch(Str, getKind() == IOKind::Outputting &&
+ (Val & Mask) == ConstVal))
Val = Val | ConstVal;
}
@@ -942,7 +955,8 @@ class IO {
bool Required, Context &Ctx) {
void *SaveInfo;
bool UseDefault;
- const bool sameAsDefault = outputting() && Val == DefaultValue;
+ const bool sameAsDefault =
+ (getKind() == IOKind::Outputting) && Val == DefaultValue;
if ( this->preflightKey(Key, Required, sameAsDefault, UseDefault,
SaveInfo) ) {
yamlize(*this, Val, Required, Ctx);
@@ -1004,7 +1018,7 @@ yamlize(IO &io, T &Val, bool, EmptyContext &Ctx) {
template <typename T>
std::enable_if_t<has_ScalarTraits<T>::value, void> yamlize(IO &io, T &Val, bool,
EmptyContext &Ctx) {
- if ( io.outputting() ) {
+ if (io.getKind() != IOKind::Inputting) {
SmallString<128> Storage;
raw_svector_ostream Buffer(Storage);
ScalarTraits<T>::output(Val, io.getContext(), Buffer);
@@ -1024,7 +1038,7 @@ std::enable_if_t<has_ScalarTraits<T>::value, void> yamlize(IO &io, T &Val, bool,
template <typename T>
std::enable_if_t<has_BlockScalarTraits<T>::value, void>
yamlize(IO &YamlIO, T &Val, bool, EmptyContext &Ctx) {
- if (YamlIO.outputting()) {
+ if (YamlIO.getKind() != IOKind::Inputting) {
std::string Storage;
raw_string_ostream Buffer(Storage);
BlockScalarTraits<T>::output(Val, YamlIO.getContext(), Buffer);
@@ -1043,7 +1057,7 @@ yamlize(IO &YamlIO, T &Val, bool, EmptyContext &Ctx) {
template <typename T>
std::enable_if_t<has_TaggedScalarTraits<T>::value, void>
yamlize(IO &io, T &Val, bool, EmptyContext &Ctx) {
- if (io.outputting()) {
+ if (io.getKind() != IOKind::Inputting) {
std::string ScalarStorage, TagStorage;
raw_string_ostream ScalarBuffer(ScalarStorage), TagBuffer(TagStorage);
TaggedScalarTraits<T>::output(Val, io.getContext(), ScalarBuffer,
@@ -1085,7 +1099,7 @@ yamlize(IO &io, T &Val, bool, Context &Ctx) {
io.beginFlowMapping();
else
io.beginMapping();
- if (io.outputting()) {
+ if (io.getKind() == IOKind::Outputting) {
std::string Err = detail::doValidate(io, Val, Ctx);
if (!Err.empty()) {
errs() << Err << "\n";
@@ -1093,7 +1107,7 @@ yamlize(IO &io, T &Val, bool, Context &Ctx) {
}
}
detail::doMapping(io, Val, Ctx);
- if (!io.outputting()) {
+ if (io.getKind() == IOKind::Inputting) {
std::string Err = detail::doValidate(io, Val, Ctx);
if (!Err.empty())
io.setError(Err);
@@ -1113,7 +1127,7 @@ yamlizeMappingEnumInput(IO &io, T &Val...
[truncated]
``````````
</details>
https://github.com/llvm/llvm-project/pull/133284
More information about the llvm-commits
mailing list