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.

Conceptually this is probably OK, but I'd like to see it run through the pre-commit testing before final ACK.


Jeff

Reply via email to