Skip to content

Commit

Permalink
[ARM] Cortex-R5 is not VFPOnlySP
Browse files Browse the repository at this point in the history
This patch fixes the error in ARM.td which stated that Cortex-R5
floating point unit can do only single precision, when it can do double as well.

Reviewers: rengolin

Subscribers: llvm-commits

Differential Revision: http://reviews.llvm.org/D10769

llvm-svn: 240799
  • Loading branch information
javedabsar committed Jun 26, 2015
1 parent c4866d2 commit bced303
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion llvm/lib/Target/ARM/ARM.td
Original file line number Diff line number Diff line change
Expand Up @@ -416,7 +416,7 @@ def : ProcessorModel<"cortex-r4f", CortexA8Model,
def : ProcessorModel<"cortex-r5", CortexA8Model,
[ProcR5, HasV7Ops, FeatureDB,
FeatureVFP3, FeatureDSPThumb2,
FeatureHasRAS, FeatureVFPOnlySP,
FeatureHasRAS,
FeatureD16, FeatureRClass]>;

// FIXME: R7 has currently the same ProcessorModel as A8 and is modelled as R5.
Expand Down
2 changes: 1 addition & 1 deletion llvm/test/CodeGen/ARM/build-attributes.ll
Original file line number Diff line number Diff line change
Expand Up @@ -1071,7 +1071,7 @@
; CORTEX-R5: .eabi_attribute 23, 3
; CORTEX-R5: .eabi_attribute 24, 1
; CORTEX-R5: .eabi_attribute 25, 1
; CORTEX-R5: .eabi_attribute 27, 1
; CORTEX-R5-NOT: .eabi_attribute 27, 1
; CORTEX-R5-NOT: .eabi_attribute 28
; CORTEX-R5-NOT: .eabi_attribute 36
; CORTEX-R5: .eabi_attribute 38, 1
Expand Down

0 comments on commit bced303

Please sign in to comment.