[Scummvm-tracker] [ScummVM] #6810: ARM: Various Crashes due to buggy ARM ASM optimizations

[Colin Snover]

#6810: ARM: Various Crashes due to buggy ARM ASM optimizations
-------------------------+--------------------
  Reporter:  SF/apoleon  |      Owner:  fuzzie
      Type:  defect      |     Status:  new
  Priority:  blocker     |  Component:  Ports
Resolution:              |   Keywords:
      Game:              |
-------------------------+--------------------
Changes (by csnover):

 * priority:  normal => blocker


Comment:

 USE_ARM_SOUND_ASM has been disabled since 2015, AFAICT PR!#625 did not
 change anything in this regard so the changes to the ARM ASM in that PR
 aren’t actually doing anything.

 It seems unnecessarily burdensome to maintain this old hand-written ARM
 assembly, which might not even be as efficient as compiled C++ code on
 modern devices with newer architectures, so I think it probably ought to
 just get removed and allow the compiler to generate appropriate machine
 code instead (since this is what is already happening anyway).

 I’m setting this to blocker to make a final decision one way or the other
 before the next release.

--
Ticket URL: <https://bugs.scummvm.org/ticket/6810#comment:11>
ScummVM <https://bugs.scummvm.org>
ScummVM