[gem5-dev] Change in gem5/gem5[develop]: misc: Replace std::conditional with std::conditional_t

2021-04-15 Thread Giacomo Travaglini (Gerrit) via gem5-dev
Giacomo Travaglini has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/44506 )


Change subject: misc: Replace std::conditional with std::conditional_t
..

misc: Replace std::conditional with std::conditional_t

Change-Id: I50d26d958d521c30b69d31426380b1e2e213a9e6
Signed-off-by: Giacomo Travaglini 
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/44506
Reviewed-by: Jason Lowe-Power 
Maintainer: Jason Lowe-Power 
Tested-by: kokoro 
---
M src/arch/amdgpu/vega/operand.hh
M src/arch/generic/vec_pred_reg.hh
M src/base/coroutine.hh
M src/base/refcnt.hh
4 files changed, 14 insertions(+), 14 deletions(-)

Approvals:
  Jason Lowe-Power: Looks good to me, approved; Looks good to me, approved
  kokoro: Regressions pass



diff --git a/src/arch/amdgpu/vega/operand.hh  
b/src/arch/amdgpu/vega/operand.hh

index 740fbb9..a4517ea 100644
--- a/src/arch/amdgpu/vega/operand.hh
+++ b/src/arch/amdgpu/vega/operand.hh
@@ -326,12 +326,12 @@
   scRegData.read();
   }

-  using VecRegCont = typename std::conditional-  VecRegContainerU64, typename  
std::conditional
+  using VecRegCont = typename std::conditional_t+  VecRegContainerU64, typename  
std::conditional_t
   == sizeof(VecElemU16), VecRegContainerU16,
-  typename std::conditional::type>::type>::type;
+  VecRegContainerU32>>>;

   /**
* whether this operand a scalar or not.
diff --git a/src/arch/generic/vec_pred_reg.hh  
b/src/arch/generic/vec_pred_reg.hh

index 46f6f2f..d156ba0 100644
--- a/src/arch/generic/vec_pred_reg.hh
+++ b/src/arch/generic/vec_pred_reg.hh
@@ -72,10 +72,10 @@

   public:
 /// Container type alias.
-using Container = typename std::conditional<
+using Container = typename std::conditional_t<
 Const,
 const VecPredRegContainer,
-VecPredRegContainer>::type;
+VecPredRegContainer>;

   protected:
 // Alias for this type
diff --git a/src/base/coroutine.hh b/src/base/coroutine.hh
index 68d2b1d..594c1e8 100644
--- a/src/base/coroutine.hh
+++ b/src/base/coroutine.hh
@@ -66,11 +66,11 @@
 // in case the channel should be void (Coroutine template parameters
 // are void. (See following ArgChannel, RetChannel typedef)
 struct Empty {};
-using ArgChannel = typename std::conditional<
-std::is_same::value, Empty, std::stack>::type;
+using ArgChannel = typename std::conditional_t<
+std::is_same::value, Empty, std::stack>;

-using RetChannel = typename std::conditional<
-std::is_same::value, Empty, std::stack>::type;
+using RetChannel = typename std::conditional_t<
+std::is_same::value, Empty, std::stack>;

   public:
 /**
diff --git a/src/base/refcnt.hh b/src/base/refcnt.hh
index 3106a6f..43a2a36 100644
--- a/src/base/refcnt.hh
+++ b/src/base/refcnt.hh
@@ -129,13 +129,13 @@
 /** Convenience aliases for const/non-const versions of T w/  
friendship. */

 /** @{ */
 static constexpr auto TisConst = std::is_const::value;
-using ConstT = typename std::conditional,
-RefCountingPtr::type>>::type;
+RefCountingPtr::type>>;
 friend ConstT;
-using NonConstT = typename std::conditional::type>,
-RefCountingPtr>::type;
+RefCountingPtr>;
 friend NonConstT;
 /** @} */
 /// The stored pointer.

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/44506
To unsubscribe, or for help writing mail filters, visit  
https://gem5-review.googlesource.com/settings


Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I50d26d958d521c30b69d31426380b1e2e213a9e6
Gerrit-Change-Number: 44506
Gerrit-PatchSet: 2
Gerrit-Owner: Giacomo Travaglini 
Gerrit-Reviewer: Bobby R. Bruce 
Gerrit-Reviewer: Gabe Black 
Gerrit-Reviewer: Giacomo Travaglini 
Gerrit-Reviewer: Jason Lowe-Power 
Gerrit-Reviewer: Jason Lowe-Power 
Gerrit-Reviewer: kokoro 
Gerrit-MessageType: merged
___
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

[gem5-dev] Change in gem5/gem5[develop]: misc: Replace std::conditional with std::conditional_t

2021-04-14 Thread Giacomo Travaglini (Gerrit) via gem5-dev
Giacomo Travaglini has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/44506 )



Change subject: misc: Replace std::conditional with std::conditional_t
..

misc: Replace std::conditional with std::conditional_t

Change-Id: I50d26d958d521c30b69d31426380b1e2e213a9e6
Signed-off-by: Giacomo Travaglini 
---
M src/arch/amdgpu/vega/operand.hh
M src/arch/generic/vec_pred_reg.hh
M src/base/coroutine.hh
M src/base/refcnt.hh
4 files changed, 14 insertions(+), 14 deletions(-)



diff --git a/src/arch/amdgpu/vega/operand.hh  
b/src/arch/amdgpu/vega/operand.hh

index 740fbb9..a4517ea 100644
--- a/src/arch/amdgpu/vega/operand.hh
+++ b/src/arch/amdgpu/vega/operand.hh
@@ -326,12 +326,12 @@
   scRegData.read();
   }

-  using VecRegCont = typename std::conditional-  VecRegContainerU64, typename  
std::conditional
+  using VecRegCont = typename std::conditional_t+  VecRegContainerU64, typename  
std::conditional_t
   == sizeof(VecElemU16), VecRegContainerU16,
-  typename std::conditional::type>::type>::type;
+  VecRegContainerU32>>>;

   /**
* whether this operand a scalar or not.
diff --git a/src/arch/generic/vec_pred_reg.hh  
b/src/arch/generic/vec_pred_reg.hh

index 46f6f2f..d156ba0 100644
--- a/src/arch/generic/vec_pred_reg.hh
+++ b/src/arch/generic/vec_pred_reg.hh
@@ -72,10 +72,10 @@

   public:
 /// Container type alias.
-using Container = typename std::conditional<
+using Container = typename std::conditional_t<
 Const,
 const VecPredRegContainer,
-VecPredRegContainer>::type;
+VecPredRegContainer>;

   protected:
 // Alias for this type
diff --git a/src/base/coroutine.hh b/src/base/coroutine.hh
index 68d2b1d..594c1e8 100644
--- a/src/base/coroutine.hh
+++ b/src/base/coroutine.hh
@@ -66,11 +66,11 @@
 // in case the channel should be void (Coroutine template parameters
 // are void. (See following ArgChannel, RetChannel typedef)
 struct Empty {};
-using ArgChannel = typename std::conditional<
-std::is_same::value, Empty, std::stack>::type;
+using ArgChannel = typename std::conditional_t<
+std::is_same::value, Empty, std::stack>;

-using RetChannel = typename std::conditional<
-std::is_same::value, Empty, std::stack>::type;
+using RetChannel = typename std::conditional_t<
+std::is_same::value, Empty, std::stack>;

   public:
 /**
diff --git a/src/base/refcnt.hh b/src/base/refcnt.hh
index 3106a6f..43a2a36 100644
--- a/src/base/refcnt.hh
+++ b/src/base/refcnt.hh
@@ -129,13 +129,13 @@
 /** Convenience aliases for const/non-const versions of T w/  
friendship. */

 /** @{ */
 static constexpr auto TisConst = std::is_const::value;
-using ConstT = typename std::conditional,
-RefCountingPtr::type>>::type;
+RefCountingPtr::type>>;
 friend ConstT;
-using NonConstT = typename std::conditional::type>,
-RefCountingPtr>::type;
+RefCountingPtr>;
 friend NonConstT;
 /** @} */
 /// The stored pointer.

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/44506
To unsubscribe, or for help writing mail filters, visit  
https://gem5-review.googlesource.com/settings


Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I50d26d958d521c30b69d31426380b1e2e213a9e6
Gerrit-Change-Number: 44506
Gerrit-PatchSet: 1
Gerrit-Owner: Giacomo Travaglini 
Gerrit-MessageType: newchange
___
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s