diff options
| author | Diana Picus <diana.picus@linaro.org> | 2017-02-24 10:35:39 +0000 |
|---|---|---|
| committer | Diana Picus <diana.picus@linaro.org> | 2017-02-24 10:35:39 +0000 |
| commit | c21d1e5d94dcab9597fa761117806de8f2b35e42 (patch) | |
| tree | 6eddcff36bd08acdb134d0c0aed9f4274d71b5ae /llvm | |
| parent | a5f1cfd1a7975f3b0dde369238d97c8d6d9d871c (diff) | |
| download | bcm5719-llvm-c21d1e5d94dcab9597fa761117806de8f2b35e42.tar.gz bcm5719-llvm-c21d1e5d94dcab9597fa761117806de8f2b35e42.zip | |
Revert "[ARM] GlobalISel: Legalize stores"
This reverts commit r296103 because the test broke on one of the bots. Sorry!
llvm-svn: 296104
Diffstat (limited to 'llvm')
| -rw-r--r-- | llvm/lib/Target/ARM/ARMLegalizerInfo.cpp | 8 | ||||
| -rw-r--r-- | llvm/test/CodeGen/ARM/GlobalISel/arm-legalizer.mir | 37 |
2 files changed, 3 insertions, 42 deletions
diff --git a/llvm/lib/Target/ARM/ARMLegalizerInfo.cpp b/llvm/lib/Target/ARM/ARMLegalizerInfo.cpp index 01752d67f9c..ceebc39e898 100644 --- a/llvm/lib/Target/ARM/ARMLegalizerInfo.cpp +++ b/llvm/lib/Target/ARM/ARMLegalizerInfo.cpp @@ -37,11 +37,9 @@ ARMLegalizerInfo::ARMLegalizerInfo(const ARMSubtarget &ST) { setAction({G_FRAME_INDEX, p0}, Legal); - for (unsigned Op : {G_LOAD, G_STORE}) { - for (auto Ty : {s1, s8, s16, s32, p0}) - setAction({Op, Ty}, Legal); - setAction({Op, 1, p0}, Legal); - } + for (auto Ty : {s1, s8, s16, s32, p0}) + setAction({G_LOAD, Ty}, Legal); + setAction({G_LOAD, 1, p0}, Legal); for (auto Ty : {s1, s8, s16, s32}) setAction({G_ADD, Ty}, Legal); diff --git a/llvm/test/CodeGen/ARM/GlobalISel/arm-legalizer.mir b/llvm/test/CodeGen/ARM/GlobalISel/arm-legalizer.mir index 6a1a58ca636..4943a7305fb 100644 --- a/llvm/test/CodeGen/ARM/GlobalISel/arm-legalizer.mir +++ b/llvm/test/CodeGen/ARM/GlobalISel/arm-legalizer.mir @@ -9,7 +9,6 @@ define void @test_load_from_stack() { ret void } define void @test_legal_loads() #0 { ret void } - define void @test_legal_stores() #0 { ret void } define void @test_fadd_s32() #0 { ret void } define void @test_fadd_s64() #0 { ret void } @@ -200,42 +199,6 @@ body: | BX_RET 14, _ ... --- -name: test_legal_stores -# CHECK-LABEL: name: test_legal_stores -legalized: false -# CHECK: legalized: true -regBankSelected: false -selected: false -tracksRegLiveness: true -registers: - - { id: 0, class: _ } - - { id: 1, class: _ } - - { id: 2, class: _ } - - { id: 3, class: _ } - - { id: 4, class: _ } - - { id: 5, class: _ } - - { id: 6, class: _ } -body: | - bb.0: - liveins: %r0, %r1, %r2, %r3 - - ; These are all legal, so we should find them unchanged in the output - ; CHECK-DAG: G_STORE {{%[0-9]+}}(s64), %0(p0) - ; CHECK-DAG: G_STORE {{%[0-9]+}}(s32), %0(p0) - ; CHECK-DAG: G_STORE {{%[0-9]+}}(s16), %0(p0) - ; CHECK-DAG: G_STORE {{%[0-9]+}}(s8), %0(p0) - ; CHECK-DAG: G_STORE {{%[0-9]+}}(s1), %0(p0) - ; CHECK-DAG: G_STORE {{%[0-9]+}}(p0), %0(p0) - %0(p0) = COPY %r0 - G_STORE %1(s64), %0(p0) :: (store 8) - G_STORE %2(s32), %0(p0) :: (store 4) - G_STORE %3(s16), %0(p0) :: (store 2) - G_STORE %4(s8), %0(p0) :: (store 1) - G_STORE %5(s1), %0(p0) :: (store 1) - G_STORE %6(p0), %0(p0) :: (store 4) - BX_RET 14, _ -... ---- name: test_fadd_s32 # CHECK-LABEL: name: test_fadd_s32 legalized: false |
