[Libre-soc-isa] [Bug 1048] OPF ISA External RFC ls011 - Fixed and Floating point LD/ST-with-update EXT2xx instructions

[bugzilla-daemon at libre-soc.org]

https://bugs.libre-soc.org/show_bug.cgi?id=1048

--- Comment #29 from Luke Kenneth Casson Leighton <lkcl at lkcl.net> ---
(In reply to Andrey Miroshnikov from comment #28)
> Removed duplicate Fixed-Point stores: stbup, stbupx, sthup, sthupx, stwup,
> stwupx, stdup, stdupx.
> https://git.libre-soc.org/?p=libreriscv.git;a=commitdiff;
> h=af645794cab13e73fa5e1f0cada344342584a00d
> 
> Added FP LD Post update section (openpower/isa/fpload)
> https://git.libre-soc.org/?p=libreriscv.git;a=commitdiff;
> h=36c2d7f2f581809e99cd3a7a82a7f175311c3c06
> 
> Added FP ST Post update section (openpower/isa/fpstore)
> https://git.libre-soc.org/?p=libreriscv.git;a=commitdiff;
> h=e9f1a5fffb382515d3cb5c6d537e60f31987c8da
> 
> The summary table also included the following instructions (which weren't
> mentioned in comment #0, but I added them for completeness):
> 
> Added Fixed LD Shifted Post update section (openpower/isa/pifixedloadshift)
> https://git.libre-soc.org/?p=libreriscv.git;a=commitdiff;
> h=517306571ec9f1bf792118acec4100c47c73f413
> 
> Added Fixed ST Shifted Post update section (openpower/isa/pifixedstoreshift)
> https://git.libre-soc.org/?p=libreriscv.git;a=commitdiff;
> h=fc2a6d3269771ffeeb423b4b0d9f6928a318b5c1
> 
> Added Floating-Point LD Shifted Post update section
> (openpower/isa/pifploadshift)
> https://git.libre-soc.org/?p=libreriscv.git;a=commitdiff;
> h=bcfc69937ef94cc45e3772906a0312cf7a8c46d2
> 
> Added Floating-Point ST Shifted Post update section
> (openpower/isa/pifpstoreshift)
> https://git.libre-soc.org/?p=libreriscv.git;a=commitdiff;
> h=a175a4756f5b4e1350d2099c3f0aae4954b337ed
> 
> 
> I just noticed that "openpower/isa/fixedload" and "openpower/isa/fixedstore"
> define instructions already present in PowerISA 3.0/1.

yes.

> This means that the two inline imports for "fixedload" and "fixedstore" are
> not required.

correct :)


> Should I remove them from ls011.mdwn (since nothing new is being added to
> the PowerISA spec)?

it would be a leeetle odd (understatement) to propose instructions
already added, but more than that, they are not Post-Update
instructions or Shifted (or both).

-- 
You are receiving this mail because:
You are on the CC list for the bug.