Skip to content

Commit

Permalink
[SPARC] Fix BooleanContents, so that select of a trunc doesn't
Browse files Browse the repository at this point in the history
eliminate the trunc.

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

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@245444 91177308-0d34-0410-b5e6-96231b3b80d8
  • Loading branch information
jyknight committed Aug 19, 2015
1 parent 468e70f commit 77c6e1a
Show file tree
Hide file tree
Showing 2 changed files with 25 additions and 0 deletions.
8 changes: 8 additions & 0 deletions lib/Target/Sparc/SparcISelLowering.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1423,6 +1423,14 @@ SparcTargetLowering::SparcTargetLowering(TargetMachine &TM,
: TargetLowering(TM), Subtarget(&STI) {
MVT PtrVT = MVT::getIntegerVT(8 * TM.getPointerSize());

// Instructions which use registers as conditionals examine all the
// bits (as does the pseudo SELECT_CC expansion). I don't think it
// matters much whether it's ZeroOrOneBooleanContent, or
// ZeroOrNegativeOneBooleanContent, so, arbitrarily choose the
// former.
setBooleanContents(ZeroOrOneBooleanContent);
setBooleanVectorContents(ZeroOrOneBooleanContent);

// Set up the register classes.
addRegisterClass(MVT::i32, &SP::IntRegsRegClass);
addRegisterClass(MVT::f32, &SP::FPRegsRegClass);
Expand Down
17 changes: 17 additions & 0 deletions test/CodeGen/SPARC/select-mask.ll
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
; RUN: llc -march=sparc < %s

;; getBooleanContents on Sparc used to claim that no bits mattered
;; other than the first for SELECT. Thus, the 'trunc' got eliminated
;; as redundant. But, cmp does NOT ignore the other bits!

; CHECK-LABEL select_mask:
; CHECK: ldub [%o0], [[R:%[goli][0-7]]]
; CHECK: and [[R]], 1, [[V:%[goli][0-7]]]
; CHECK: cmp [[V]], 0
define i32 @select_mask(i8* %this) {
entry:
%bf.load2 = load i8, i8* %this, align 4
%bf.cast5 = trunc i8 %bf.load2 to i1
%cond = select i1 %bf.cast5, i32 2, i32 0
ret i32 %cond
}

0 comments on commit 77c6e1a

Please sign in to comment.