[Libre-soc-bugs] [Bug 979] Implement C-based Power ISA decoder compiler

bugzilla-daemon at libre-soc.org bugzilla-daemon at libre-soc.org
Sat Sep 2 12:14:18 BST 2023


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

--- Comment #18 from Luke Kenneth Casson Leighton <lkcl at lkcl.net> ---
(In reply to Dmitry Selyutin from comment #15)
> I think you remember how binutils did these tricks, eh?
> 
> #define TO TBR + 1
> #define DUI TO
> #define SVme TO
> #define SVG TO

oh dear...

> We don't care about names, at all, other than for documentation purposes.

internally, no. externally, yes.  i.e. "as an API user", it should
be bloody obvious, when someone looks at the spec, that "operand N
is under Form M which oh look it's really obvious that that links
this header file line Y to section A.B.C of the spec" 

therefore can you please make sure that the header-file and/or structs contain
the Form as part of the "unique identification"?  something like:

   X_FORM_L0
   X_FORM_L1
   ...

i have very specifically ensured that "L" (which is a nightmare
candidate that i have been warning IBM about for 3 years) as an
example is unique... *only* if you prefix it with the Form, by
renaming conflicting occurrences of "L" to "L0 L1 L2 L3".

i really want this to work with c, and not to critically depend
on c++ namespaces.

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


More information about the libre-soc-bugs mailing list