RTM re-nits for xabort

Summary:
[Folly] RTM re-nits for `xabort`.

* Remove param pack.
* Always noreturn.

Reviewed By: nbronson

Differential Revision: D17117350

fbshipit-source-id: b16056b1f02956d334da4494c689a4b988a961d4

folly/synchronization/detail/Hardware.cpp

@@ -119,10 +119,7 @@ FOLLY_RTM_DISABLED_NORETURN static bool rtmTestFunc() {
 #endif
 }
 
-template <size_t... I>
-FOLLY_RTM_DISABLED_NORETURN FOLLY_DISABLE_SANITIZERS
-    FOLLY_ALWAYS_INLINE static void
-    rtmAbortFunc_(std::index_sequence<I...>, uint8_t status) {
+[[noreturn]] FOLLY_DISABLE_SANITIZERS static void rtmAbortFunc(uint8_t status) {
 #if FOLLY_RTM_SUPPORT
   switch (status) {
 #define FOLLY_RTM_ABORT_ONE(z, n, text) \
@@ -139,10 +136,6 @@ FOLLY_RTM_DISABLED_NORETURN FOLLY_DISABLE_SANITIZERS
 #endif
 }
 
-FOLLY_DISABLE_SANITIZERS static void rtmAbortFunc(uint8_t status) {
-  rtmAbortFunc_(std::make_index_sequence<256u>{}, status);
-}
-
 namespace detail {
 
 unsigned rtmBeginDisabled() {