Boris Shingarov has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/42943 )
Change subject: arch-power: Refactor CR field generation
......................................................................
arch-power: Refactor CR field generation
This splits the existing makeCRField utility into signed
and unsigned variants to help callers avoid confusion.
The CR bit union is also used to clean up the underlying
bit setting logic.
Change-Id: I2aa6ec0666d2bc5096eb6c775cc47f2a5a0621ee
Signed-off-by: Sandipan Das <sandi...@linux.ibm.com>
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/42943
Tested-by: kokoro <noreply+kok...@google.com>
Reviewed-by: Gabe Black <gabe.bl...@gmail.com>
Maintainer: Bobby R. Bruce <bbr...@ucdavis.edu>
---
M src/arch/power/insts/integer.hh
M src/arch/power/isa/decoder.isa
M src/arch/power/isa/formats/integer.isa
3 files changed, 21 insertions(+), 19 deletions(-)
Approvals:
Gabe Black: Looks good to me, approved
Bobby R. Bruce: Looks good to me, approved
kokoro: Regressions pass
diff --git a/src/arch/power/insts/integer.hh
b/src/arch/power/insts/integer.hh
index 1c9b1cc..1771fea 100644
--- a/src/arch/power/insts/integer.hh
+++ b/src/arch/power/insts/integer.hh
@@ -65,28 +65,30 @@
/* Compute the CR (condition register) field using signed comparison */
inline uint32_t
- makeCRField(int32_t a, int32_t b, uint32_t xerSO) const
+ makeCRFieldSigned(int64_t a, int64_t b, bool so) const
{
- uint32_t c = xerSO;
+ Cr cr = 0;
- /* We've pre-shifted the immediate values here */
- if (a < b) { c += 0x8; }
- else if (a > b) { c += 0x4; }
- else { c += 0x2; }
- return c;
+ if (a < b) { cr.cr0.lt = 1; }
+ else if (a > b) { cr.cr0.gt = 1; }
+ else { cr.cr0.eq = 1; }
+ if (so) { cr.cr0.so = 1; }
+
+ return cr.cr0;
}
/* Compute the CR (condition register) field using unsigned comparison
*/
inline uint32_t
- makeCRField(uint32_t a, uint32_t b, uint32_t xerSO) const
+ makeCRFieldUnsigned(uint64_t a, uint64_t b, bool so) const
{
- uint32_t c = xerSO;
+ Cr cr = 0;
- /* We've pre-shifted the immediate values here */
- if (a < b) { c += 0x8; }
- else if (a > b) { c += 0x4; }
- else { c += 0x2; }
- return c;
+ if (a < b) { cr.cr0.lt = 1; }
+ else if (a > b) { cr.cr0.gt = 1; }
+ else { cr.cr0.eq = 1; }
+ if (so) { cr.cr0.so = 1; }
+
+ return cr.cr0;
}
std::string generateDisassembly(
diff --git a/src/arch/power/isa/decoder.isa b/src/arch/power/isa/decoder.isa
index f32861b..7f22d8c 100644
--- a/src/arch/power/isa/decoder.isa
+++ b/src/arch/power/isa/decoder.isa
@@ -39,12 +39,12 @@
format IntImmOp {
10: cmpli({{
Xer xer = XER;
- uint32_t cr = makeCRField(Ra, (uint32_t)uimm, xer.so);
+ uint32_t cr = makeCRFieldUnsigned(Ra_uw, uimm, xer.so);
CR = insertCRField(CR, BF, cr);
}});
11: cmpi({{
Xer xer = XER;
- uint32_t cr = makeCRField(Ra_sw, (int32_t)imm, xer.so);
+ uint32_t cr = makeCRFieldSigned(Ra_sw, imm, xer.so);
CR = insertCRField(CR, BF, cr);
}});
}
@@ -250,12 +250,12 @@
format IntOp {
0: cmp({{
Xer xer = XER;
- uint32_t cr = makeCRField(Ra_sw, Rb_sw, xer.so);
+ uint32_t cr = makeCRFieldSigned(Ra_sw, Rb_sw, xer.so);
CR = insertCRField(CR, BF, cr);
}});
32: cmpl({{
Xer xer = XER;
- uint32_t cr = makeCRField(Ra, Rb, xer.so);
+ uint32_t cr = makeCRFieldUnsigned(Ra_uw, Rb_uw, xer.so);
CR = insertCRField(CR, BF, cr);
}});
144: mtcrf({{
diff --git a/src/arch/power/isa/formats/integer.isa
b/src/arch/power/isa/formats/integer.isa
index 50badce..0ed0bf02 100644
--- a/src/arch/power/isa/formats/integer.isa
+++ b/src/arch/power/isa/formats/integer.isa
@@ -77,7 +77,7 @@
computeCR0Code = '''
Cr cr = CR;
- cr.cr0 = makeCRField((int32_t)%(result)s, (int32_t)0, xer.so);
+ cr.cr0 = makeCRFieldSigned(%(result)s, 0, xer.so);
CR = cr;
'''
2 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the
submitted one.
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/42943
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: I2aa6ec0666d2bc5096eb6c775cc47f2a5a0621ee
Gerrit-Change-Number: 42943
Gerrit-PatchSet: 5
Gerrit-Owner: Sandipan Das <sandi...@linux.ibm.com>
Gerrit-Reviewer: Bobby R. Bruce <bbr...@ucdavis.edu>
Gerrit-Reviewer: Boris Shingarov <shinga...@gmail.com>
Gerrit-Reviewer: Gabe Black <gabe.bl...@gmail.com>
Gerrit-Reviewer: kokoro <noreply+kok...@google.com>
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
