[Libre-soc-bugs] [Bug 712] New: introduce helper class which aggregates all helpers

[bugzilla-daemon at libre-soc.org]

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

            Bug ID: 712
           Summary: introduce helper class which aggregates all helpers
           Product: Libre-SOC's first SoC
           Version: unspecified
          Hardware: PC
                OS: Linux
            Status: CONFIRMED
          Severity: enhancement
          Priority: ---
         Component: Source Code
          Assignee: lkcl at lkcl.net
          Reporter: dmitry.selyutin at 3mdeb.com
                CC: libre-soc-bugs at lists.libre-soc.org
   NLnet milestone: ---

src/openpower/decoder/helpers.py currently has a lot of functions here and
there. Putting aside massive global scope pollution, it also introduces another
problem: it's completely unusable when some of the functions must be updated so
that they take an advantage of XLEN variable. Since XLEN is not part of global
scope, it's impossible for instructions to make use of XLEN.
The solution is to distribute everything as part of a class which is
XLEN-aware. Once functions are converted into methods, they can access class
internals. The created class is supposed to be inherited by ISACaller.

The works include the following steps:

* create a stub class with methods which call the routines from the global
scope
* teach decoder/pseudo/parser.py to deal with these methods instead of global
routines
* once parser calls all routines from the class, move all the gory
implementation details from global scope into the methods

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