[oe] [meta-oe][PATCH] luajit: mips and mipsel are 32bit targets too

[Andre McCurdy]

On Wed, Jan 25, 2017 at 12:58 AM, André Draszik <git at andred.net> wrote:
> On Tue, 2017-01-24 at 11:19 -0800, Andre McCurdy wrote:
>> When cross compiling, the word size of the compiler set via HOST_CC
>> must match the word size of the target. That's achieved by appending
>> "-m32" to BUILD_CC_ARCH if the target word size is known to be 32bits.
>>
>> Unfortunately the current list of over-rides (powerpc, x86 and arm)
>> does not cover all cases. Add mips and mipsel to the list too (which
>> is still not enough to cover all targets or corner cases such as x32,
>> but better than before).
>>
>> Signed-off-by: Andre McCurdy <armccurdy at gmail.com>
>> ---
>>  meta-oe/recipes-devtools/luajit/luajit_2.0.4.bb | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/meta-oe/recipes-devtools/luajit/luajit_2.0.4.bb b/meta-
>> oe/recipes-devtools/luajit/luajit_2.0.4.bb
>> index 964dc1d..2049846 100644
>> --- a/meta-oe/recipes-devtools/luajit/luajit_2.0.4.bb
>> +++ b/meta-oe/recipes-devtools/luajit/luajit_2.0.4.bb
>> @@ -24,6 +24,8 @@ BBCLASSEXTEND = "native"
>>  BUILD_CC_ARCH_append_powerpc = ' -m32'
>>  BUILD_CC_ARCH_append_x86 = ' -m32'
>>  BUILD_CC_ARCH_append_arm = ' -m32'
>> +BUILD_CC_ARCH_append_mips = ' -m32'
>> +BUILD_CC_ARCH_append_mipsel = ' -m32'
>
> I suspect this should be using:
>
> BUILD_CC_ARCH_append_mipsarchn32 = ' -m32'
> BUILD_CC_ARCH_append_mipsarcho32 = ' -m32'

That wouldn't work so well when backported to krogoth and morty (which
is where I personally need the fix).

But maybe there's a more generic way to answer the question "is the
target 32bit?" from within a recipe without needing to keep a list of
over-rides up to date?

> to also cover mips r6?
>
>
> Cheers,
> Andre'
>
>
>
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel at lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel