copy with modifications was at: https://libre-soc.org/openpower/sv/int_fp_mv_reduced_insn_count/ now merged back into: https://libre-soc.org/openpower/sv/int_fp_mv/ I created a reduced instruction list as well as a bunch of notes. * DONE: rewrite the rest of that file to match the new naming scheme. * DONE: get everyone to look at it and agree on what we want to do * markos and jacob have reviewed it and want to keep old scheme * DONE: lkcl reviewed and we decided to use the new scheme but with the old scheme as instruction aliases, with a note that we considered the old scheme. * DONE: if we decide we want the reduced instruction scheme, merge it back into int_fp_mv.mdwn (must wait till agreement is reached)
markos looked at it and decided he likes the old unreduced scheme better: https://libre-soc.org/irclog/%23libre-soc.2023-03-09.log.html#t2023-03-09T09:00:58 i agree.
(In reply to Jacob Lifshay from comment #1) > markos looked at it and decided he likes the old unreduced scheme better: > https://libre-soc.org/irclog/%23libre-soc.2023-03-09.log.html#t2023-03-09T09: > 00:58 the answer was no, 6 months ago, and remains no. https://libre-soc.org/irclog/%23libre-soc.2023-03-14.log.html#t2023-03-14T10:36:45
I rewrote int_fp_mv_reduced_insn_count and merged the changes into int_fp_mv, so I'm marking this resolved. we can mostly copy sections into the RFC, I already did the hard work of writing the full pseudo-code and stuff (if anyone spots any bugs, do let me know!). https://git.libre-soc.org/?p=libreriscv.git;a=blob;f=openpower/sv/int_fp_mv.mdwn;h=5c0afa2dee68c71e1e8788f72185770c12e7ef71;hb=16580f9f1a53f6f4a37701fcb397ef11286f9cb3 commit 16580f9f1a53f6f4a37701fcb397ef11286f9cb3 (HEAD -> master, origin/master, origin/HEAD) Author: Jacob Lifshay <programmerjake@gmail.com> Date: Wed Mar 15 01:16:57 2023 -0700 move int_fp_mv_reduced_insn_count.mdwn over int_fp_mv.mdwn commit 648046411a9ae7399bdd9fb1b736ad7c157bcb5d Author: Jacob Lifshay <programmerjake@gmail.com> Date: Wed Mar 15 01:16:15 2023 -0700 copy changes over from int_fp_mv.mdwn commit e5cb91f114797017c2981ae49f4270d362614aaa Author: Jacob Lifshay <programmerjake@gmail.com> Date: Wed Mar 15 01:12:22 2023 -0700 delete now-unnecessary section on experimental naming schemes commit be35db55be1a155afdb9681a0ff28f5d20352fcb Author: Jacob Lifshay <programmerjake@gmail.com> Date: Wed Mar 15 01:11:51 2023 -0700 add note on considered alternative naming schemes commit 5b3e3929d3102937ee4bd00d40c86ebe1b037519 Author: Jacob Lifshay <programmerjake@gmail.com> Date: Wed Mar 15 01:01:12 2023 -0700 rewrite int_fp_mv_reduced_insn_count to account for reduced instructions changes