Hey,
I didn't really take my time to review the patch, but I can see that you implemented a function for counting leading zeros, which isn't really necessary, because theres a builtin for that (__builtin_clzll).
Also, we are using GitHub's pull request system to submit changes, so that would be a better way show us what you've done.
GitHub repository: https://github.com/cbaylis/luajit-aarch64/tree/aarch64-v2.1-new
Regards, Stefan
On Tue, 2 Aug 2016 22:53:34 +0000 "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