On 5/20/24 5:32 PM, Vineet Gupta wrote:
This is testsuite clean however there's a dwarf quirk which I want to run by the experts. The test that was tripping CI has following fragment: Before patch | After Patch ------------------------------------------------------ li t0,-4096 | addi sp,s0,-2048 addi t0,t0,560 | .cfi_def_cfa 2, 2048 <- #1 add sp,s0,t0 | addi sp,sp,-1488 .cfi_def_cfa 2, 3536 | .cfi_def_cfa_offset 3536 <- #2 addi sp,sp,1504 | addi sp,sp,1504 .cfi_def_cfa_offset 2032 | .cfi_def_cfa_offset 2032 <- #3 The dwarf insn #1 and #3 seem ok, however #2 seems dubious to me. --- This is continuing on the prev patch in function epilogue expansion. gcc/ChangeLog: * config/riscv/riscv.cc (riscv_expand_epilogue): Handle offset being sum of two S12.
OK. jeff