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

Reply via email to