================
@@ -81,14 +89,20 @@ template <bool Signed> class IntegralAP final {
   }
 
   explicit operator bool() const { return !V.isZero(); }
-  explicit operator int8_t() const { return truncateCast<int8_t>(V); }
-  explicit operator uint8_t() const { return truncateCast<uint8_t>(V); }
-  explicit operator int16_t() const { return truncateCast<int16_t>(V); }
-  explicit operator uint16_t() const { return truncateCast<uint16_t>(V); }
-  explicit operator int32_t() const { return truncateCast<int32_t>(V); }
-  explicit operator uint32_t() const { return truncateCast<uint32_t>(V); }
-  explicit operator int64_t() const { return truncateCast<int64_t>(V); }
-  explicit operator uint64_t() const { return truncateCast<uint64_t>(V); }
+  explicit operator int8_t() const { return truncateCast<int8_t, Signed>(V); }
+  explicit operator uint8_t() const { return truncateCast<uint8_t, Signed>(V); 
}
+  explicit operator int16_t() const { return truncateCast<int16_t, Signed>(V); 
}
+  explicit operator uint16_t() const {
+    return truncateCast<uint16_t, Signed>(V);
+  }
+  explicit operator int32_t() const { return truncateCast<int32_t, Signed>(V); 
}
+  explicit operator uint32_t() const {
+    return truncateCast<uint32_t, Signed>(V);
+  }
+  explicit operator int64_t() const { return truncateCast<int64_t, Signed>(V); 
}
+  explicit operator uint64_t() const {
+    return truncateCast<uint64_t, Signed>(V);
+  }
----------------
AaronBallman wrote:

This might be a terrible idea, but I wish we could remove the duplication with 
something along the lines of:
```
template <typename Ty, typename = std::enable_if_t<std::is_integral_v<Ty>>>
explicit operator Ty() const { return truncateCast<Ty, Signed>(V); }
```
instead of having to repeat ourselves so much. CC @cor3ntin 

https://github.com/llvm/llvm-project/pull/69911
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to