Re: Looks like a minor bug in dis_arm64.lua

Hi Abhishek,

Both asm_conv and regname functions have been already fixed. Please clone the following repository: https://github.com/cbaylis/luajit-aarch64/tree/aarch64-v2.1-new

Stefan

On Tue, 19 Jul 2016 01:05:10 +0000 "Dikshit, Abhishek" Abhishek.Dikshit@cavium.com wrote:

Hi I was trying the -jdump=bitmsr option in ARM64 and I was getting a failure in VM. I made a small modification. Please review and let me know if the modification is correct.

---

diff --git a/src/jit/dis_arm64.lua b/src/jit/dis_arm64.lua index c1b6201..b39b825 100644 --- a/src/jit/dis_arm64.lua +++ b/src/jit/dis_arm64.lua @@ -1176,8 +1176,8 @@ end

-- Return register name for RID. local function regname(r)

• if r < 32 then return map_reg.x[r] end
• return map_reg.d[r-32]

• if r < 32 then return map_regs.x[r] end
• return map_regs.d[r-32]

end

-- Public module functions.

Thanks Abhishek Dikshit