[llvm] deb9520 - [ADT] Simplify check_options with std::conjunction (NFC) (#159509)

via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 18 08:55:37 PDT 2025


Author: Kazu Hirata
Date: 2025-09-18T08:55:33-07:00
New Revision: deb95201b2795b74f6269e5b8e126fb2ba8fb380

URL: https://github.com/llvm/llvm-project/commit/deb95201b2795b74f6269e5b8e126fb2ba8fb380
DIFF: https://github.com/llvm/llvm-project/commit/deb95201b2795b74f6269e5b8e126fb2ba8fb380.diff

LOG: [ADT] Simplify check_options with std::conjunction (NFC) (#159509)

This patch replaces the recursion with std::conjunction for
readability and brevity.

Added: 
    

Modified: 
    llvm/include/llvm/ADT/ilist_node_options.h

Removed: 
    


################################################################################
diff  --git a/llvm/include/llvm/ADT/ilist_node_options.h b/llvm/include/llvm/ADT/ilist_node_options.h
index 143195aa9c647..003d5dabce897 100644
--- a/llvm/include/llvm/ADT/ilist_node_options.h
+++ b/llvm/include/llvm/ADT/ilist_node_options.h
@@ -145,12 +145,8 @@ struct is_valid_option<ilist_parent<ParentTy>> : std::true_type {};
 /// Check whether options are valid.
 ///
 /// The conjunction of \a is_valid_option on each individual option.
-template <class... Options> struct check_options;
-template <> struct check_options<> : std::true_type {};
-template <class Option1, class... Options>
-struct check_options<Option1, Options...>
-    : std::bool_constant<is_valid_option<Option1>::value &&
-                         check_options<Options...>::value> {};
+template <class... Options>
+struct check_options : std::conjunction<is_valid_option<Options>...> {};
 
 /// Traits for options for \a ilist_node.
 ///


        


More information about the llvm-commits mailing list