]> granicus.if.org Git - clang/commit
[ARM] Add support for the s,j,x,N,O inline asm constraints
authorDavid Candler <david.candler@arm.com>
Thu, 5 Sep 2019 15:17:25 +0000 (15:17 +0000)
committerDavid Candler <david.candler@arm.com>
Thu, 5 Sep 2019 15:17:25 +0000 (15:17 +0000)
commit57e3a8e7783fd84776c37603b4e9ed35a7539f2d
tree6b358334151cb8e8d7e65ba3cefe6753f79c6bfc
parent280c4e53349a35cbe54732d7063390ff82a07cc5
[ARM] Add support for the s,j,x,N,O inline asm constraints

A number of inline assembly constraints are currently supported by LLVM, but rejected as invalid by Clang:

Target independent constraints:

s: An integer constant, but allowing only relocatable values

ARM specific constraints:

j: An immediate integer between 0 and 65535 (valid for MOVW)
x: A 32, 64, or 128-bit floating-point/SIMD register: s0-s15, d0-d7, or q0-q3
N: An immediate integer between 0 and 31 (Thumb1 only)
O: An immediate integer which is a multiple of 4 between -508 and 508. (Thumb1 only)

This patch adds support to Clang for the missing constraints along with some checks to ensure that the constraints are used with the correct target and Thumb mode, and that immediates are within valid ranges (at least where possible). The constraints are already implemented in LLVM, but just a couple of minor corrections to checks (V8M Baseline includes MOVW so should work with 'j', 'N' and 'O' shouldn't be valid in Thumb2) so that Clang and LLVM are in line with each other and the documentation.

Differential Revision: https://reviews.llvm.org/D65863

Change-Id: I18076619e319bac35fbb60f590c069145c9d9a0a

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@371079 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Basic/Targets/ARM.cpp
test/Sema/arm_inline_asm_constraints.c [new file with mode: 0644]