Float to Integer instructions cause assertion: "Got an I2F instruction in an ExecutionPipe that does not source the integer RF"

[danbone]

All of the float to integer instructions cause this assertion to fire within the execute pipe:

riscv-perf-model/core/ExecutePipe.cpp

Lines 148 to 150 in d745493

	if(SPARTA_EXPECT_FALSE(ex_inst->getPipe() == InstArchInfo::TargetPipe::I2F)) {
	sparta_assert(reg_file_ == core_types::RegFile::RF_INTEGER,
	"Got an I2F instruction in an ExecutionPipe that does not source the integer RF");

If the intention is that these instructions use the i2f target pipe (not a new f2i type), then I think the code just needs to be changed to handle f2i, and the assertion removed.

Test trace:

[
    {
        "mnemonic": "fcvt.w.s",
        "rs1": 1,
        "rs2": 2,
        "rd": 3
    }
]

[ghost]

Yep, appears to not have been tested. Thanks for letting us know.

Better to support an f2i target to allow for future pipe assignments (maybe f2i is handled by FP pipes and i2f is handled by IEX pipes).

A tweak to your JSON input:

[
    {
        "mnemonic": "fcvt.w.s",
        "fs1": 1,
        "rd": 3
    }
]

frm is ignored...

Got a fix I'm testing now.

[ghost]

PR up. Please take a look.