more cleanup

Author: SchrodingerZhu

libc/src/__support/threads/callonce.h

@@ -25,12 +25,14 @@
 namespace LIBC_NAMESPACE_DECL {
 
 // Common definitions
-using CallOnceCallback = void(void);
 namespace callonce_impl {
-int callonce_slowpath(CallOnceFlag *flag, CallOnceCallback *callback);
+template <class CallOnceCallback>
+[[gnu::noinline]] int callonce_slowpath(CallOnceFlag *flag,
+                                        CallOnceCallback callback);
 } // namespace callonce_impl
 
-LIBC_INLINE int callonce(CallOnceFlag *flag, CallOnceCallback *callback) {
+template <class CallOnceCallback>
+LIBC_INLINE int callonce(CallOnceFlag *flag, CallOnceCallback callback) {
   if (LIBC_LIKELY(callonce_impl::callonce_fastpath(flag)))
     return 0;
 

libc/src/pthread/pthread_once.cpp

@@ -17,8 +17,7 @@ namespace LIBC_NAMESPACE_DECL {
 
 LLVM_LIBC_FUNCTION(int, pthread_once,
                    (pthread_once_t * flag, __pthread_once_func_t func)) {
-  return callonce(reinterpret_cast<CallOnceFlag *>(flag),
-                  reinterpret_cast<CallOnceCallback *>(func));
+  return callonce(reinterpret_cast<CallOnceFlag *>(flag), func);
 }
 
 } // namespace LIBC_NAMESPACE_DECL

libc/src/threads/call_once.cpp

@@ -17,8 +17,7 @@ namespace LIBC_NAMESPACE_DECL {
 
 LLVM_LIBC_FUNCTION(void, call_once,
                    (once_flag * flag, __call_once_func_t func)) {
-  callonce(reinterpret_cast<CallOnceFlag *>(flag),
-           reinterpret_cast<CallOnceCallback *>(func));
+  callonce(reinterpret_cast<CallOnceFlag *>(flag), func);
 }
 
 } // namespace LIBC_NAMESPACE_DECL