Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ARM assembler rejects LDRT/LDRHT/LDRBT with "error: instruction requires: thumb2" #18094

Closed
llvmbot opened this issue Oct 29, 2013 · 3 comments
Closed

ARM assembler rejects LDRT/LDRHT/LDRBT with "error: instruction requires: thumb2" #18094

llvmbot opened this issue Oct 29, 2013 · 3 comments
Labels
bugzilla Issues migrated from bugzilla

Comments

@llvmbot
Copy link
Collaborator

llvmbot commented Oct 29, 2013

Bugzilla Link 17720
Resolution FIXED
Resolved on Jun 19, 2014 22:18
Version trunk
OS Linux
Blocks #19300
Reporter LLVM Bugzilla Contributor
CC @compnerd,@rengolin

Extended Description

The integrated ARM assembler in Clang rejects LDRT/LDRHT/LDRBT with "error: instruction requires: thumb2" when assembling for ARMv7-A.

Steps to reproduce for LDRT:

$ cat > test.s <<END
.syntax unified
.text
.arm
LDRT r0, [r1]
END
$ /opt/llvm/bin/clang -target armv7a-none-eabi -integrated-as test.s

Actual results:
'+soft-float-abi' is not a recognized feature for this target (ignoring feature)
test.s:4:1: error: instruction requires: thumb2
LDRT r0, [r1]
^

Expected results: assemble LDRT instruction without error, since it has been a valid ARM instruction since ARMv4, long before Thumb2 first appeared.

Version and platform:
$ /opt/llvm/bin/clang --version
clang version 3.4 (trunk 193589)
Target: x86_64-unknown-linux-gnu
Thread model: posix
@compnerd
Copy link
Member

This seems to be fixed. I think that this should be fixed by the changes that I did to handle the GNU extensions (SVN r198914). The provided test case passes with current ToT. Marking the bug as fixed.

@rengolin
Copy link
Member

mentioned in issue #19300

@agners
Copy link
Contributor

agners commented Nov 27, 2021

mentioned in issue llvm/llvm-bugzilla-archive#45070

@rengolin rengolin mentioned this issue Feb 21, 2014
Open
@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 9, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bugzilla Issues migrated from bugzilla
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@agners @compnerd @rengolin @llvmbot