On 7/7/24 11:29 PM, Fei Gao wrote:
On 2024-07-06 22:15 Jeff Law <jeffreya...@gmail.com> wrote:
On 7/5/24 3:56 AM, Fei Gao wrote:
According to Zc-1.0.4-3.pdf from
https://github.com/riscvarchive/riscv-code-size-reduction/releases/tag/v1.0.4-3
The rule is that:
1. C always implies Zca
2. C+F implies Zcf (RV32 only)
3. C+D implies Zcd
gcc/ChangeLog:
* common/config/riscv/riscv-common.cc:
c implies zca, and conditionally zcf & zcd.
gcc/testsuite/ChangeLog:
* gcc.target/riscv/attribute-15.c: adapt TC.
* gcc.target/riscv/attribute-18.c: likewise.
* gcc.target/riscv/pr110696.c: likewise.
* gcc.target/riscv/rvv/base/abi-callee-saved-1-zcmp.c: likewise.
* gcc.target/riscv/rvv/base/abi-callee-saved-2-zcmp.c: likewise.
* gcc.target/riscv/rvv/base/pr114352-1.c: likewise.
* gcc.target/riscv/rvv/base/pr114352-3.c: likewise.
* gcc.target/riscv/arch-39.c: New test.
* gcc.target/riscv/arch-40.c: New test.
This failed to apply in the RISC-V tester. It's not clear to me why.
What I would suggest is get 1/2 installed, then wait for the tester to
pick up 1/2, then resubmit 2/2.
Thanks for your reveiw and advice.
Patch 1/2 has been installed and I will resend patch 2/2 in a few days
to retrigger CI.
I would think you could send it tomorrow. The post-commit CI runs every
8 hrs I think to update the baseline. Given your patch went in this
morning, I would expect the baseline to move forward past patch 1/2 by
the time you start your day tomorrow.
jeff
