Skip to content

Commit a82e63a

Browse files
committed
Fix clang compatibility of jsimd_arm_neon.S
Clang only supports Unified Assembler Language (and not pre-UAL syntax). This is intentional. https://llvm.org/bugs/show_bug.cgi?id=23507 http://infocenter.arm.com/help/index.jsp?topic=/com.arm.doc.dui0473c/BABJIHGJ.html This means clang expects "strbeq" here. Unfortunately, it seems that my version of gcc only supports "streqb". So I think we need both variations here.
1 parent 0d61e80 commit a82e63a

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

simd/jsimd_arm_neon.S

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2457,7 +2457,11 @@ asm_function jsimd_h2v1_fancy_upsample_neon
24572457
strb \TMP, [\BUFFER, #1]!
24582458
cmp \TMP, #0xff
24592459
/*it eq*/
2460+
#if defined(__clang__)
2461+
strbeq \ZERO, [\BUFFER, #1]!
2462+
#else
24602463
streqb \ZERO, [\BUFFER, #1]!
2464+
#endif
24612465
.endm
24622466

24632467
.macro put_bits PUT_BUFFER, PUT_BITS, CODE, SIZE

0 commit comments

Comments
 (0)