Hi Lehua, > I think you are rigth, I would like to remove the `-mcmodel=medany` option and > relax assert from `__riscv_save/restore_4` to `__riscv_save/restore_(3|4)` to > let > this testcase not brittle on any -mcmodel. Then I'm also going to add another > testcase (I dont known how to run -march=rv32imafc and -march=rv64imafc on > the same testcase) that uses -march=rv64imafc.
Thanks, ideally we'd have a common include that contains the code but I don't think we need to stress on that now. At a later point, we'd ideally have no rv32/rv64 ever in the options (apart from specific bugs) and just run the test suite for both so only one file would be needed. But we don't have that yet, so LGTM. Regards Robin