derek/gem5
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

cpu: Fix style in inst_res.hh.

Browse Source 
Change-Id: Ifc6d980f5dc0cec8b6ac58e4fb390ed49e3e1a39
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/49123
Reviewed-by: Bobby R. Bruce <bbruce@ucdavis.edu>
Maintainer: Bobby R. Bruce <bbruce@ucdavis.edu>
Tested-by: kokoro <noreply+kokoro@google.com>
This commit is contained in:
Gabe Black
2021-08-08 23:09:42 -07:00
parent d8da636bbb
commit 27c186f436
1 changed files with 34 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files

54
src/cpu/inst_res.hh
View File

@@ -78,7 +78,8 @@ class InstResult
InstResult(const InstResult &) = default;
/** Scalar result from scalar. */
template<typename T>
explicit InstResult(T i, const ResultType& t) : type(t) {
explicit InstResult(T i, const ResultType& t) : type(t)
{
static_assert(std::is_integral_v<T> ^ std::is_floating_point_v<T>,
"Parameter type is neither integral nor fp, or it is both");
if (std::is_integral_v<T>) {
@@ -89,62 +90,75 @@ class InstResult
}
/** Vector result. */
explicit InstResult(const TheISA::VecRegContainer& v, const ResultType& t)
: type(t) { result.vector = v; }
: type(t)
{
result.vector = v;
}
/** Predicate result. */
explicit InstResult(const TheISA::VecPredRegContainer& v,
const ResultType& t)
: type(t) { result.pred = v; }
: type(t)
{
result.pred = v;
}
InstResult& operator=(const InstResult& that) {
InstResult&
operator=(const InstResult& that)
{
type = that.type;
switch (type) {
/* Given that misc regs are not written to, there may be invalids in
* the result stack. */
case ResultType::Invalid:
// Given that misc regs are not written to, there may be invalids in
//the result stack.
case ResultType::Invalid:
break;
case ResultType::Scalar:
case ResultType::Scalar:
result.integer = that.result.integer;
break;
case ResultType::VecElem:
case ResultType::VecElem:
result.vecElem = that.result.vecElem;
break;
case ResultType::VecReg:
case ResultType::VecReg:
result.vector = that.result.vector;
break;
case ResultType::VecPredReg:
case ResultType::VecPredReg:
result.pred = that.result.pred;
break;
default:
default:
panic("Assigning result from unknown result type");
break;
}
return *this;
}
/**
* Result comparison
* Two invalid results always differ.
*/
bool operator==(const InstResult& that) const {
bool
operator==(const InstResult& that) const
{
if (this->type != that.type)
return false;
switch (type) {
case ResultType::Scalar:
case ResultType::Scalar:
return result.integer == that.result.integer;
case ResultType::VecElem:
case ResultType::VecElem:
return result.vecElem == that.result.vecElem;
case ResultType::VecReg:
case ResultType::VecReg:
return result.vector == that.result.vector;
case ResultType::VecPredReg:
case ResultType::VecPredReg:
return result.pred == that.result.pred;
case ResultType::Invalid:
case ResultType::Invalid:
return false;
default:
default:
panic("Unknown type of result: %d\n", (int)type);
}
}
bool operator!=(const InstResult& that) const {
bool
operator!=(const InstResult& that) const
{
return !operator==(that);
}