libretro / libretro/parallel-n64

replace __ARM_NEON__ with HAVE_NEON ?

Open
#195 28 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
C
Stars
407
Forks
151
Avg merge
11h 53m
Merged PRs (30d)
11

Description

Would be nice to replace all those ARM_NEON with HAVE_NEON. From what i understood in gizmo98's fork, it seems like neon is the reason why the core built by this repo won't run properly on rpi2, and you have to fully disable neon optimization to build a usable mupen64-libretro core on rpi2.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Search the repository for ARM_NEON and inspect how HAVE_NEON is defined in the build configuration. Reproduce or examine the Raspberry Pi 2 build path, then verify that disabling NEON produces a usable mupen64-libretro core without breaking NEON-enabled builds.

Written by the indexing model from the issue text.

Assessment

Tech stack
c
Domain
build-system, game-dev
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.