On 11/12/2023 21:42, Thomas Schwinge wrote:
Hi Andre!

On 2023-10-16T16:03:26+0100, "Andre Vieira (lists)" 
<andre.simoesdiasvie...@arm.com> wrote:
Just a minor update to the patch, I had missed the libgomp testsuite, so
had to make some adjustments there too.

Unfortunately, there appear to be a number of DejaGnu directive errors in
your test case changes -- do you not see those in your testing?

I hadn't seen those... I wonder whether they don't show up if you do dg-cmp-results with just one -v, I have binned the build, but I'll rerun it and double check, may need to use '-v -v' instead.

Thanks for letting me know.
..., and the following change also doesn't look quite right:

--- a/libgomp/testsuite/libgomp.fortran/declare-simd-1.f90
+++ b/libgomp/testsuite/libgomp.fortran/declare-simd-1.f90
@@ -1,5 +1,5 @@
  ! { dg-do run { target vect_simd_clones } }
-! { dg-options "-fno-inline" }
+! { dg-options "-fno-inline -cpp -D__aarch64__" }


Yeah, that needs a target selector. Thanks!

Reply via email to