The code snippet for this test was returning 1 if power10
instructions executed correctly. It should return 0 if the
test passes.

Approved offline by Segher with slight change. Will
push after posting.


        * lib/target-supports.exp (check_power10_hw_available):
        Return 0 for passing test.
---
 gcc/testsuite/lib/target-supports.exp | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/gcc/testsuite/lib/target-supports.exp 
b/gcc/testsuite/lib/target-supports.exp
index 4bdcaef1132..848cb96aec4 100644
--- a/gcc/testsuite/lib/target-supports.exp
+++ b/gcc/testsuite/lib/target-supports.exp
@@ -2226,7 +2226,9 @@ proc check_power10_hw_available { } {
                /* Set e first and use +r to check if pli actually works.  */
                long e = -1;
                asm ("pli %0,%1" : "+r" (e) : "n" (0x12345));
-               return (e == 0x12345);
+               if (e == 0x12345)
+                 return 0;
+               return 1;
            }
        } "-mcpu=power10"
     }]
-- 
2.17.1

Reply via email to