From 025df3b8270083666c5e091810216d1bc9e7fe6e Mon Sep 17 00:00:00 2001 From: Eli Friedman Date: Tue, 14 May 2019 21:44:54 +0000 Subject: [PATCH] [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 --- openmp/runtime/src/z_Linux_asm.S | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/openmp/runtime/src/z_Linux_asm.S b/openmp/runtime/src/z_Linux_asm.S index 1197864..39bf427 100644 --- a/openmp/runtime/src/z_Linux_asm.S +++ b/openmp/runtime/src/z_Linux_asm.S @@ -1243,7 +1243,7 @@ __tid = 8 orr w9, wzr, #1 add w9, w9, w3, lsr #1 - sub sp, sp, w9, lsl #4 + sub sp, sp, w9, uxtw #4 mov x11, sp mov x8, x0 -- 2.7.4