On 10/30/24 14:13, Jeff Law wrote:
> On 10/29/24 6:11 PM, Vineet Gupta wrote:
>> When bisecting for ICE in PR/117353, commit 771256bcb9dd ("RISC-V: Emit
>> costs for
>> bool and stepped const vectors") uncovered yet another latent issue (first
>> noted [1])
>>
>> [1] https://github.com/patrick-rivos/gcc-postcommit-ci/issues/1625
>>
>> This patch fixes some of the fortran regressions from that report.
>>
>> Fixes 71a5ac6703d1 ("RISC-V: Support interleave vector with different step
>> sequence")
>>
>> rv64imafdcv_zvl256b_zba_zbb_zbs_zicond/lp64d/medlow
>> | # of unexpected case / # of unique unexpected
>> case
>> | gcc | g++ | gfortran |
>> | 392 / 108 | 7 / 3 | 91 / 24 |
>> | 392 / 108 | 7 / 3 | 67 / 12 |
>>
>> gcc/ChangeLog:
>>
>> * config/riscv/riscv-v.cc (expand_const_vector): Use IOR op.
>>
>> gcc/testsuite/ChangeLog:
>>
>> * gcc.target/riscv/rvv/autovec/slp-interleave-5.c: New test.
>>
>> Signed-off-by: Vineet Gupta <vine...@rivosinc.com>
> OK. Though I don't see any improvements in the pre-commit tester.
Yeah I just checked with Edwin and Patrick about this.
The issue is triggered by zvl256b while pre-commit doesn't specify anything (so
default 128 ?) and gfortran tests are generic so don't specify zvl
toggles either.
My newly added test should pass but that won't be reported as it doesn't
"resolve" any existing failure.
Thx,
-Vineet
