[llvm] [Support] Simplify yamlizeMappingEnumInput (NFC) (PR #137537)
via llvm-commits
llvm-commits at lists.llvm.org
Sun Apr 27 11:14:41 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-llvm-support
Author: Kazu Hirata (kazutakahirata)
<details>
<summary>Changes</summary>
We can use "constexpt if" to combine the two variants of functions.
---
Full diff: https://github.com/llvm/llvm-project/pull/137537.diff
1 Files Affected:
- (modified) llvm/include/llvm/Support/YAMLTraits.h (+11-15)
``````````diff
diff --git a/llvm/include/llvm/Support/YAMLTraits.h b/llvm/include/llvm/Support/YAMLTraits.h
index e707a445012b5..21829b906967e 100644
--- a/llvm/include/llvm/Support/YAMLTraits.h
+++ b/llvm/include/llvm/Support/YAMLTraits.h
@@ -1105,22 +1105,18 @@ yamlize(IO &io, T &Val, bool, Context &Ctx) {
}
template <typename T, typename Context>
-std::enable_if_t<!has_MappingEnumInputTraits<T, Context>::value, bool>
-yamlizeMappingEnumInput(IO &io, T &Val) {
- return false;
-}
-
-template <typename T, typename Context>
-std::enable_if_t<has_MappingEnumInputTraits<T, Context>::value, bool>
-yamlizeMappingEnumInput(IO &io, T &Val) {
- if (io.outputting())
- return false;
+bool yamlizeMappingEnumInput(IO &io, T &Val) {
+ if constexpr (has_MappingEnumInputTraits<T, Context>::value) {
+ if (io.outputting())
+ return false;
- io.beginEnumScalar();
- MappingTraits<T>::enumInput(io, Val);
- bool Matched = !io.matchEnumFallback();
- io.endEnumScalar();
- return Matched;
+ io.beginEnumScalar();
+ MappingTraits<T>::enumInput(io, Val);
+ bool Matched = !io.matchEnumFallback();
+ io.endEnumScalar();
+ return Matched;
+ }
+ return false;
}
template <typename T, typename Context>
``````````
</details>
https://github.com/llvm/llvm-project/pull/137537
More information about the llvm-commits
mailing list