Hi Joern,
I'm seeing new failures introduced by this patch
(9464e72bcc9123b619215af8cfef491772a3ebd9).
On rv64gcv:
FAIL: gcc.dg/pr90263.c scan-assembler memcpy
FAIL: gfortran.fortran-torture/execute/intrinsic_count.f90 execution,
-O2 -fomit-frame-pointer -finline-functions -funroll-loops
Debug log for intrinsic_count.f90:
spawn riscv64-unknown-linux-gnu-run
/scratch/tc-testing/tc-410-break/build/build-gcc-linux-stage2/gcc/testsuite/gfortran9/intrinsic_count.x
STOP 2
FAIL: gfortran.fortran-torture/execute/intrinsic_count.f90 execution,
-O2 -fomit-frame-pointer -finline-functions -funroll-loops
It's worth noting that intrinsic_count.f90 had failures prior to this
patch for other option combinations:
FAIL: gfortran.fortran-torture/execute/intrinsic_count.f90 execution, -O2
FAIL: gfortran.fortran-torture/execute/intrinsic_count.f90 execution,
-O2 -fbounds-check
FAIL: gfortran.fortran-torture/execute/intrinsic_count.f90 execution,
-O2 -fomit-frame-pointer -finline-functions
Thanks,
Patrick
On 10/1/23 19:43, Joern Rennecke wrote:
On Tue, 15 Aug 2023 at 15:06, Jeff Law <jeffreya...@gmail.com> wrote:
>
On 8/15/23 03:16, juzhe.zh...@rivai.ai wrote:
The new patch looks reasonable to me now. Thanks for fixing it.
Could you append testcase after finishing test infrastructure ?
I prefer this patch with testcase after infrastructure.
So let's call this an ACK, but ask that Joern not commit until the
testsuite bits are in place.
Beyond the adding of tests, the patch needed some changes because of the
Refactoring of emit_{vlmax,nonvlmax}_xxx functions .
Attached is the committed version.
