Conversation
|
Thank you for your support of arm ecology, we will review your code and make sure there is no problem to merge it. |
| #elif defined(HAVE_NEON) | ||
| __asm__ __volatile__("ldr w0, %w2 \n\t" | ||
| "bic %w0,w0,%w1 \n\t" | ||
| : "=r"(r) |
There was a problem hiding this comment.
This is our optimization, it is not recommended to modify.
|
|
||
| u32 reach0, reach1, reach2, reach3; | ||
| u64a ptr = unaligned_load_u64a(itPtr); | ||
|
|
There was a problem hiding this comment.
This is also our optimization, it is not recommended to modify.
| #define CRC32CB(crc, value) __asm__("crc32cb %w[c], %w[c], %w[v]":[c]"+r"(crc):[v]"r"(value)) | ||
| #define CRC_WORD 8 | ||
| #define CRC_TYPE u64a | ||
| static really_inline |
There was a problem hiding this comment.
It is not recommended to modify for all of the “HAVE_NEON” branches.
| } | ||
| #else | ||
| if (check_neon()) { | ||
| return HS_SUCCESS; |
There was a problem hiding this comment.
This check is needed for arm platform.
| union RepeatControl *repeat_ctrl = ctx->repeat_ctrl + info->ctrlIndex; | ||
| char *repeat_state = ctx->repeat_state + info->stateOffset; | ||
|
|
||
| if (unlikely(e->trigger == LIMEX_TRIGGER_POS)) { |
There was a problem hiding this comment.
These branch predictions can improve the performance and are not recommended to be deleted.
|
kylin seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
3 participants
No description provided.