Skip to content

Commit 025df3b

Browse files
committed
[OpenMP][AArch64] Fix compile with LLVM trunk.
The code is currently using the ambiguous instruction "sub sp, sp, w9, lsl #4". The ARM reference manual says this isn't valid, and it's not clear whether it's supposed to mean uxtw or uxtx. It doesn't matter which instruction we use here, since the high bits of the operand are zero anyway, so I arbitrarily choose uxtw, to preserve the register name. See https://reviews.llvm.org/D60840 for the LLVM patch. Differential Revision: https://reviews.llvm.org/D61770 llvm-svn: 360711
1 parent c9dd299 commit 025df3b

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

openmp/runtime/src/z_Linux_asm.S

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1243,7 +1243,7 @@ __tid = 8
12431243

12441244
orr w9, wzr, #1
12451245
add w9, w9, w3, lsr #1
1246-
sub sp, sp, w9, lsl #4
1246+
sub sp, sp, w9, uxtw #4
12471247
mov x11, sp
12481248

12491249
mov x8, x0

0 commit comments

Comments
 (0)