1 files changed, 83 insertions, 0 deletions
diff --git a/results/classifier/108/other/1625987 b/results/classifier/108/other/1625987
new file mode 100644
index 000000000..63142c09d
--- /dev/null
+++ b/results/classifier/108/other/1625987
@@ -0,0 +1,83 @@
+semantic: 0.627
+device: 0.580
+PID: 0.567
+performance: 0.557
+socket: 0.484
+graphic: 0.418
+network: 0.397
+permissions: 0.387
+boot: 0.340
+KVM: 0.333
+vnc: 0.319
+other: 0.288
+files: 0.244
+debug: 0.159
+
+target-arm/translate-a64.c:2028: possible coding error ?
+
+target-arm/translate-a64.c:2028:37: warning: ?: using integer constants in boolean context [-Wint-in-bool-context]
+
+Source code is
+
+        bool iss_sf = opc == 0 ? 32 : 64;
+
+Maybe better code
+
+        bool iss_sf = (opc == 0) ? 32 : 64;
+
+This is clearly a bug, but your suggested change won't deal with the problem, which is that we're trying to set a bool so the ? 32 : 64 construct is just wrong.
+
+
+On 22 September 2016 at 02:53, Peter Maydell <email address hidden> wrote:
+> This is clearly a bug, but your suggested change won't deal with the
+> problem, which is that we're trying to set a bool so the ? 32 : 64
+> construct is just wrong.
+
+> Bug description:
+>   target-arm/translate-a64.c:2028:37: warning: ?: using integer
+>   constants in boolean context [-Wint-in-bool-context]
+>
+>   Source code is
+>
+>           bool iss_sf = opc == 0 ? 32 : 64;
+
+Edgar, did you want to look at a fix for this? It was introduced
+in your commit aaa1f954d4 adding syndrome info for loads and stores.
+
+thanks
+-- PMM
+
+
+On Thu, Sep 29, 2016 at 06:40:53PM -0700, Peter Maydell wrote:
+> On 22 September 2016 at 02:53, Peter Maydell <email address hidden> wrote:
+> > This is clearly a bug, but your suggested change won't deal with the
+> > problem, which is that we're trying to set a bool so the ? 32 : 64
+> > construct is just wrong.
+> 
+> > Bug description:
+> >   target-arm/translate-a64.c:2028:37: warning: ?: using integer
+> >   constants in boolean context [-Wint-in-bool-context]
+> >
+> >   Source code is
+> >
+> >           bool iss_sf = opc == 0 ? 32 : 64;
+> 
+> Edgar, did you want to look at a fix for this? It was introduced
+> in your commit aaa1f954d4 adding syndrome info for loads and stores.
+
+Hi Peter,
+
+Yes, I've just posted a fix to the list.
+It should have been:
+
+bool iss_sf = opc == 0 ? false : true;
+
+Cheers,
+Edgar
+
+
+Now fixed in master, commit 173ff58580b383a7841.
+
+
+Released with v2.8
+