From: Ahmed Bougacha Date: Fri, 16 Sep 2016 14:44:54 +0000 (+0000) Subject: [AArch64][GlobalISel] Test default regbank mapping for G_ICMP. X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=07ca84e98a3fc255d83b0bef01ece703d5f8e95a;p=llvm [AArch64][GlobalISel] Test default regbank mapping for G_ICMP. Also relax a RegisterBankInfo verifier check that's incompatible with 1-bit mappings. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@281735 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/CodeGen/GlobalISel/RegisterBankInfo.cpp b/lib/CodeGen/GlobalISel/RegisterBankInfo.cpp index 20015ae82fd..91681fb7a40 100644 --- a/lib/CodeGen/GlobalISel/RegisterBankInfo.cpp +++ b/lib/CodeGen/GlobalISel/RegisterBankInfo.cpp @@ -390,7 +390,7 @@ void RegisterBankInfo::PartialMapping::dump() const { bool RegisterBankInfo::PartialMapping::verify() const { assert(RegBank && "Register bank not set"); assert(Length && "Empty mapping"); - assert((StartIdx < getHighBitIdx()) && "Overflow, switch to APInt?"); + assert((StartIdx <= getHighBitIdx()) && "Overflow, switch to APInt?"); // Check if the minimum width fits into RegBank. assert(RegBank->getSize() >= Length && "Register bank too small for Mask"); return true; diff --git a/test/CodeGen/AArch64/GlobalISel/regbankselect-default.mir b/test/CodeGen/AArch64/GlobalISel/regbankselect-default.mir index 12edd40dd44..9fbe25e50fe 100644 --- a/test/CodeGen/AArch64/GlobalISel/regbankselect-default.mir +++ b/test/CodeGen/AArch64/GlobalISel/regbankselect-default.mir @@ -35,6 +35,9 @@ define void @test_constant_s32() { ret void } define void @test_constant_p0() { ret void } + define void @test_icmp_s32() { ret void } + define void @test_icmp_p0() { ret void } + define void @test_frame_index_p0() { %ptr0 = alloca i64 ret void @@ -493,6 +496,44 @@ body: | %0(p0) = G_CONSTANT 0 ... +--- +# CHECK-LABEL: name: test_icmp_s32 +name: test_icmp_s32 +legalized: true +# CHECK: registers: +# CHECK: - { id: 0, class: gpr } +# CHECK: - { id: 1, class: gpr } +registers: + - { id: 0, class: _ } + - { id: 1, class: _ } +body: | + bb.0: + liveins: %w0 + ; CHECK: %0(s32) = COPY %w0 + ; CHECK: %1(s1) = G_ICMP intpred(ne), %0(s32), %0 + %0(s32) = COPY %w0 + %1(s1) = G_ICMP intpred(ne), %0, %0 +... + +--- +# CHECK-LABEL: name: test_icmp_p0 +name: test_icmp_p0 +legalized: true +# CHECK: registers: +# CHECK: - { id: 0, class: gpr } +# CHECK: - { id: 1, class: gpr } +registers: + - { id: 0, class: _ } + - { id: 1, class: _ } +body: | + bb.0: + liveins: %x0 + ; CHECK: %0(p0) = COPY %x0 + ; CHECK: %1(s1) = G_ICMP intpred(ne), %0(p0), %0 + %0(p0) = COPY %x0 + %1(s1) = G_ICMP intpred(ne), %0, %0 +... + --- # CHECK-LABEL: name: test_frame_index_p0 name: test_frame_index_p0