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

arch-arm: Fixing implicit fallthrough build errors

Browse Source 
2c242d6 introduced implicit-fallthrough errors when building against
ARM.

Added "default: return new Unknown(machInst);" to offending switch
statements; please verify this is the corret behavior

Signed-off-by: Ryan Gambord

Change-Id: I5f5e3661ec562d4a3b2699e07d1195e6877ff959
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/17071
Reviewed-by: Giacomo Travaglini <giacomo.travaglini@arm.com>
Maintainer: Giacomo Travaglini <giacomo.travaglini@arm.com>
This commit is contained in:
Ryan Gambord
2019-03-11 04:04:30 -07:00
parent 29f68cc7ba
commit 6d67e86a1b
1 changed files with 5 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/arch/arm/isa/formats/fp.isa
View File

@@ -2101,6 +2101,7 @@ let {{
} else {
return new VcvtaFpUIntD(machInst, vdInt, vm);
}
default: return new Unknown(machInst);
}
case 0x1:
switch(size) {
@@ -2121,6 +2122,7 @@ let {{
} else {
return new VcvtnFpUIntD(machInst, vdInt, vm);
}
default: return new Unknown(machInst);
}
case 0x2:
switch(size) {
@@ -2141,6 +2143,7 @@ let {{
} else {
return new VcvtpFpUIntD(machInst, vdInt, vm);
}
default: return new Unknown(machInst);
}
case 0x3:
switch(size) {
@@ -2161,7 +2164,9 @@ let {{
} else {
return new VcvtmFpUIntD(machInst, vdInt, vm);
}
default: return new Unknown(machInst);
}
default: return new Unknown(machInst);
}
}
} else if (b31_b24 && !b23 && b11_b9 && !op3 && b4){