Hello Curtis Dunham,

I'd like you to do a code review. Please visit

    https://gem5-review.googlesource.com/2900

to review the following change.


Change subject: arch-sparc: Fix wrong indentation causing warnings for gcc 6
......................................................................

arch-sparc: Fix wrong indentation causing warnings for gcc 6

Change-Id: I94e15ae79f0e73692d882f62fd2b7bf45cf0c841
Reviewed-by: Curtis Dunham <[email protected]>
---
M src/arch/sparc/isa/base.isa
M src/arch/sparc/isa/formats/mem/util.isa
2 files changed, 5 insertions(+), 3 deletions(-)



diff --git a/src/arch/sparc/isa/base.isa b/src/arch/sparc/isa/base.isa
index 8a964cb..29d2250 100644
--- a/src/arch/sparc/isa/base.isa
+++ b/src/arch/sparc/isa/base.isa
@@ -456,9 +456,10 @@
             // just print the first dest... if there's a second one,
             // it's generally implicit
             if (_numDestRegs > 0) {
-                if (_numSrcRegs > 0)
+                if (_numSrcRegs > 0) {
                     ss << ",";
-                    printReg(ss, _destRegIdx[0]);
+                }
+                printReg(ss, _destRegIdx[0]);
             }

             return ss.str();
diff --git a/src/arch/sparc/isa/formats/mem/util.isa b/src/arch/sparc/isa/formats/mem/util.isa
index a5c64dd..0a07f55 100644
--- a/src/arch/sparc/isa/formats/mem/util.isa
+++ b/src/arch/sparc/isa/formats/mem/util.isa
@@ -326,8 +326,9 @@
     '''

     TruncateEA = '''
-        if (!FullSystem)
+        if (!FullSystem) {
             EA = Pstate.am ? EA<31:0> : EA;
+        }
     '''
 }};


--
To view, visit https://gem5-review.googlesource.com/2900
To unsubscribe, visit https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I94e15ae79f0e73692d882f62fd2b7bf45cf0c841
Gerrit-Change-Number: 2900
Gerrit-PatchSet: 1
Gerrit-Owner: Nikos Nikoleris <[email protected]>
Gerrit-Reviewer: Curtis Dunham <[email protected]>
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to