[Libre-soc-isa] [Bug 685] Implement XLEN-ification for BCD instructions in base OpenPower ISA

[bugzilla-daemon at libre-soc.org]

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

--- Comment #5 from Jacob Lifshay <programmerjake at gmail.com> ---
(In reply to Luke Kenneth Casson Leighton from comment #3)
> https://git.libre-soc.org/?p=openpower-isa.git;a=commitdiff;
> h=096b8d60b6500ec8af98c2442ebb0f31066b0bcb
> 
> this depends on pia.  as i explained, pia has to go, therefore
> adding more dependence on it means more work down the line to
> do.

I think we should keep the dependence on pia, since if/when we replace it, I'm
planning on ending up with something that has the same Python API, so it'll be
a drop-in replacement. Until then, we *need* something that is tested against a
native Power9 cpu, pia is by far the easiest route to do that. The additional
work required to port pia caused by this I estimate to be around 30min -- see
the pia commit:
https://git.libre-soc.org/?p=power-instruction-analyzer.git;a=commitdiff;h=95fdd1c4edbd91c0a02b772ba02aa2045101d2b0

> can you please therefore remove the dependence on pia.

that would be several days more work, to get testing against a power9 cpu, that
code would likely just get thrown out again if/when pia is ported to non-Rust.

If we're going to invest that time it would be better to spend it writing a pia
replacement.

So, can you please just accept the new tests for now, and if we're going to
spend a bunch of extra time, let me spend it rewriting pia in Python or C or
something.

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