RFC: JamVM force interpreter -marm on ARM Linux.

[Xerxes Rånby]

On 2011-03-13 01:05, Xerxes Rånby wrote:
> On 2011-03-11 15:31, Xerxes Rånby wrote:
>> On 2011-03-11 13:46, Robert Lougher wrote:
>>>>> On 11.03.2011 12:52, Xerxes Rånby wrote:
>>>>>> Hi
>>>>>>
>>>>>> JamVM currently fail to run on ARM Linux systems where gcc by
>>>>>> default optimize for Thumb2.
>>>>>> This can be fixed by overriding the JamVM CFLAGS to skip thumb by
>>>>>> enforcing the use of -marm -g -O2 .
>>> I've been unwell for a few days, so apologies for my recent silence.
>>>
>>> I believe it's only the interpreter which needs to be built as ARM.
>>>
>>> There is already a need to set a flag for the interpreter
>>> (interp_cflags) so this might work:
>>>
>>> arm*-*-linux*) host_cpu=arm host_os=linux interp_cflags=-marm ;;
>>>
>>> The place where interp_cflags is currently set (line 180,
>>> enable_int_inlining) will also need to be changed to:
>>>
>>> interp_cflags="$interp_cflags -fno-reorder-blocks"
>>>
>>>
>>> Assuming it is only the interpreter that needs ARM, this should work.
>>> However, I'm unable to test this until tonight at the earliest.
>>>
>>> Rob.
>>
>> I have attached a new patch that implements this using Roberts
>> instructions.
>> It works nicely on my setup!
>>
>
> Hi
>
> I have committed your implementation to
> http://icedtea.classpath.org/hg/icedtea6/rev/0ca79255e13e
>
> Ok to push to Icedtea6-1.10 release branch?
>
> Cheers
> Xerxes

No need to push to Icedtea6-1.10

This have been submitted upstream.
When we recive the next jamvm code drop then we can remove the following 
patches:

patches/jamvm/mips64el.patch
patches/jamvm/arm-interp_cflags-marm.patch

Cheers
Xerxes