Hi

I've had a brief read through, and I've noticed the following:
. please remove the spurious Makefile changes
. there are several unrelated changes here. Please make them into separate patches
. if you look at asm_cnew in lj_asm_x86.h, you can see that there are significant differences for GC_64. Does your implementation match these?
. the emit_isk13 changes need to be a separate patch, and make sure that the VIXL licence conditions are followed

It would be easier to review the changes if you submitted a github pull request. It's fine to add a comment to say that it's only for review and/or post a link on the list to ask for comments.

Thanks
Charles


On 2 August 2016 at 23:53, Dikshit, Abhishek <Abhishek.Dikshit@cavium.com> wrote:

Hi All

I am new to both luajit and ARM. I have made the changes for the functions asm_tbar, asm_cnew and emit_isk13 for arm64 based on my limited understanding. I have attached the diffs. Please review the diffs and share any suggestions you have. The changes are mostly based on arm32 code for corresponding functions.

 

The diffs are corresponding to test case in the attached email.

Thanks

Abhishek Dikshit



---------- Forwarded message ----------
From: Ryan Arnold <ryan.arnold@linaro.org>
To: "Dikshit, Abhishek" <Abhishek.Dikshit@cavium.com>
Cc: "luajit@lists.linaro.org" <luajit@lists.linaro.org>
Date: Wed, 20 Jul 2016 18:34:35 +0000
Subject: Re: Looking forward to contribute.
As far as I've heard the FFI IR work has not currently started.  Here's the minimal testcase that Charles identified that will expose the FFI IR:
local ffi = require("ffi")
local typ = ffi.typeof("uint64_t")
x=0LL
for i=1,100 do x=ffi.cast(typ, i) end
print (x

On Wed, Jul 20, 2016 at 1:30 PM, Dikshit, Abhishek <Abhishek.Dikshit@cavium.com> wrote:

Hi All,

I am looking forward to contribute in development of LuaJIT for ARM64.

Please feel free to assign work to me even if it is a minor bug fix.

 

Thanks

Abhishek Dikshit


_______________________________________________
luajit mailing list
luajit@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/luajit




--

Ryan S. Arnold | Linaro Toolchain Engineering Manager

ryan.arnold@linaro.org | ryanarn on #linaro-tcwg @ freenode.irc.net

T: +1-612-424-1861


_______________________________________________
luajit mailing list
luajit@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/luajit