when dec.decoded is opcode_pcadda, there's a data width mismatch on the assignment (11 bit vs 15 bit)

[DiegoRosales]

On the following line, there's a mismatch on the data width.
I've just learned about this project so I don't know what this exactly does, but it throws errors.

fppa-pdk-emulator-vhdl/hdl/pdk14.vhd

Line 565 in 9dc957e

npc <= IPC + signextend15(A);

I don't see that function (signextend15) being used anywhere else. Can it be reduced to 11 bits like so?

  function signextend11(a: in wordtype) return unsigned is
    variable r: unsigned(10 downto 0);
  begin
    r(7 downto 0) := a;
    r(10 downto 8) := (others => a(7));
    return r;
  end signextend11;

Thanks!

[alvieboy]

Hi Diego, Indeed, that function should be for 11 bits, not for 15 (15 is used in PDK15). Let me fix it. And thanks for the report. Best regards, Alvaro Lopes

…

On 20/05/2020 19:35, Diego Rosales wrote: On the following line, there's a mismatch on the data width. I've just learned about this project so I don't know what this exactly does, but it throws errors. https://github.com/free-pdk/fppa-pdk-emulator-vhdl/blob/9dc957eda0118cdd4e63b515ca8774a14693c89b/hdl/pdk14.vhd#L565 I don't see that function (signextend15) being used anywhere else. Can it be reduced to 11 bits like so? function signextend11(a: in wordtype) return unsigned is variable r: unsigned(10 downto 0); begin r(7 downto 0) := a; r(10 downto 8) := (others => a(7)); return r; end signextend11; Thanks! — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#1>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AABDYMJH3OANBPKADFLIH3DRSQPFZANCNFSM4NGGVUGQ>.