[libcxx-commits] [PATCH] D149554: [libc++][test] Selects proper C++23 field.

[Mark de Wever via Phabricator via libcxx-commits]

Mordante marked an inline comment as done.
Mordante added a comment.

Thanks for the review!



================
Comment at: libcxx/test/tools/clang_tidy_checks/uglify_attributes.cpp:33-45
+template <class T>
+bool CPlusPlus23(const T& lang_opts)
+  requires(!std::same_as<decltype(T::CPlusPlus2b), void>)
+{
+  return lang_opts.CPlusPlus2b;
+}
+
----------------
philnik wrote:
> This should also work.
I expected a simpler solution was indeed possible. Thanks for the suggestion.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D149554/new/

https://reviews.llvm.org/D149554