maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Use more sensible type for flags in asms. PR 5570. 

Patch by Sylve`re Teissier (sorry, ASCII only).

llvm-svn: 91988
This commit is contained in:
Dale Johannesen 2009-12-23 07:32:51 +00:00
parent a7bcbde814
commit a864a67185
3 changed files with 31 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
View File

@ -5326,13 +5326,11 @@ void RegsForValue::AddInlineAsmOperands(unsigned Code,
bool HasMatching,unsigned MatchingIdx,
SelectionDAG &DAG, unsigned Order,
std::vector<SDValue> &Ops) const {
EVT IntPtrTy = DAG.getTargetLoweringInfo().getPointerTy();
assert(Regs.size() < (1 << 13) && "Too many inline asm outputs!");
unsigned Flag = Code | (Regs.size() << 3);
if (HasMatching)
Flag |= 0x80000000 | (MatchingIdx << 16);
SDValue Res = DAG.getTargetConstant(Flag, IntPtrTy);
SDValue Res = DAG.getTargetConstant(Flag, MVT::i32);
Ops.push_back(Res);
if (DisableScheduling)

3
llvm/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp
View File

@ -1182,9 +1182,8 @@ SelectInlineAsmMemoryOperands(std::vector<SDValue> &Ops) {
}
// Add this to the output node.
EVT IntPtrTy = TLI.getPointerTy();
Ops.push_back(CurDAG->getTargetConstant(4/*MEM*/ | (SelOps.size()<< 3),
IntPtrTy));
MVT::i32));
Ops.insert(Ops.end(), SelOps.begin(), SelOps.end());
i += 2;
}

29
llvm/test/CodeGen/MSP430/2009-12-22-InlineAsm.ll Normal file
View File

@ -0,0 +1,29 @@
; RUN: llc < %s
; PR 5570
; ModuleID = 'test.c'
target datalayout = "e-p:16:8:8-i8:8:8-i16:8:8-i32:8:8-n8:16"
target triple = "msp430-unknown-unknown"
@buf = common global [10 x i8] zeroinitializer, align 1 ; <[10 x i8]*> [#uses=2]
define i16 @main() noreturn nounwind {
entry:
%0 = tail call i8* asm "", "=r,0"(i8* getelementptr inbounds ([10 x i8]* @buf, i16 0, i16 0)) nounwind ; <i8*> [#uses=1]
%sub.ptr = getelementptr inbounds i8* %0, i16 1 ; <i8*> [#uses=1]
%sub.ptr.lhs.cast = ptrtoint i8* %sub.ptr to i16 ; <i16> [#uses=1]
%sub.ptr.sub = sub i16 %sub.ptr.lhs.cast, ptrtoint ([10 x i8]* @buf to i16) ; <i16> [#uses=1]
%cmp = icmp eq i16 %sub.ptr.sub, 1 ; <i1> [#uses=1]
br i1 %cmp, label %bar.exit, label %if.then.i
if.then.i: ; preds = %entry
tail call void @abort() nounwind
br label %bar.exit
bar.exit: ; preds = %entry, %if.then.i
tail call void @exit(i16 0) nounwind
unreachable
}
declare void @exit(i16) noreturn
declare void @abort()