[clang] [Tooling] Simplify a use of std::is_same (NFC) (PR #160237)

via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 22 22:44:18 PDT 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: Kazu Hirata (kazutakahirata)

<details>
<summary>Changes</summary>

std::is_same<...>::type is the same as std::is_same<...>.  This patch
removes the redundant ::type at the end.


---
Full diff: https://github.com/llvm/llvm-project/pull/160237.diff


1 Files Affected:

- (modified) clang/include/clang/Tooling/Refactoring/RefactoringOptionVisitor.h (+5-5) 


``````````diff
diff --git a/clang/include/clang/Tooling/Refactoring/RefactoringOptionVisitor.h b/clang/include/clang/Tooling/Refactoring/RefactoringOptionVisitor.h
index 3234b0976a8e7..ed2aa55c99279 100644
--- a/clang/include/clang/Tooling/Refactoring/RefactoringOptionVisitor.h
+++ b/clang/include/clang/Tooling/Refactoring/RefactoringOptionVisitor.h
@@ -37,11 +37,11 @@ namespace internal {
 template <typename T> struct HasHandle {
 private:
   template <typename ClassT>
-  static auto check(ClassT *) -> typename std::is_same<
-      decltype(std::declval<RefactoringOptionVisitor>().visit(
-          std::declval<RefactoringOption>(),
-          *std::declval<std::optional<T> *>())),
-      void>::type;
+  static auto check(ClassT *)
+      -> std::is_same<decltype(std::declval<RefactoringOptionVisitor>().visit(
+                          std::declval<RefactoringOption>(),
+                          *std::declval<std::optional<T> *>())),
+                      void>;
 
   template <typename> static std::false_type check(...);
 

``````````

</details>


https://github.com/llvm/llvm-project/pull/160237


More information about the cfe-commits mailing list