Skip to content

Processors: Add support for the Cell Broadband Engine #7888

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kukrimate
Copy link

This patch tries to add some level of support for the Cell Broadband Engine processor.

  • Most of this is the SPR list from the public Cell BE documentation
  • Also an attempt was made to fix a minor decompiler issue where volatile registers with a rename attribute break decompilation. The logic where rename overrides the name is modelled after the Java code that is doing similar parsing.
  • #4563 looks also useful.

…ibutes()

Without this volatile registers with a rename attribute will cause the
decompiler to use a different name for a renamed register, and will
cause the decompilation to fail.

Signed-off-by: Mate Kukri <[email protected]>
Most SPRs are marked as volatile, while their value in most cases
is strictly not volatile. However changes to many SPR bits have observable
side-effects, hence elliding writes (and to a lesser extent reads)
will result in a less then desirable decompilation.

Signed-off-by: Mate Kukri <[email protected]>
@luizfernandonb
Copy link

Did you add support for PPU or both PPU and SPU?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants