From: Dan Gohman Date: Tue, 17 Mar 2009 02:45:40 +0000 (+0000) Subject: Recognize bswapl as bswap too. X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=9626447e70a3c8dc88b09f84414ee416a39518b1;p=oota-llvm.git Recognize bswapl as bswap too. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@67072 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Target/X86/X86TargetAsmInfo.cpp b/lib/Target/X86/X86TargetAsmInfo.cpp index f3faa7022cd..f1d97a38932 100644 --- a/lib/Target/X86/X86TargetAsmInfo.cpp +++ b/lib/Target/X86/X86TargetAsmInfo.cpp @@ -411,8 +411,11 @@ bool X86TargetAsmInfo::ExpandInlineAsm(CallInst *CI) const { // bswap $0 if (AsmPieces.size() == 2 && - (AsmPieces[0] == "bswap" || AsmPieces[0] == "bswapq") && - (AsmPieces[1] == "$0" || AsmPieces[1] == "${0:q}")) { + (AsmPieces[0] == "bswap" || + AsmPieces[0] == "bswapq" || + AsmPieces[0] == "bswapl") && + (AsmPieces[1] == "$0" || + AsmPieces[1] == "${0:q}")) { // No need to check constraints, nothing other than the equivalent of // "=r,0" would be valid here. return LowerToBSwap(CI); diff --git a/test/CodeGen/X86/bswap-inline-asm.ll b/test/CodeGen/X86/bswap-inline-asm.ll index c44d845c832..91f8310361a 100644 --- a/test/CodeGen/X86/bswap-inline-asm.ll +++ b/test/CodeGen/X86/bswap-inline-asm.ll @@ -1,6 +1,7 @@ ; RUN: llvm-as < %s | llc -march=x86-64 > %t ; RUN: not grep APP %t ; RUN: grep bswapq %t | count 2 +; RUN: grep bswapl %t | count 1 define i64 @foo(i64 %x) nounwind { %asmtmp = tail call i64 asm "bswap $0", "=r,0,~{dirflag},~{fpsr},~{flags}"(i64 %x) nounwind @@ -10,3 +11,7 @@ define i64 @bar(i64 %x) nounwind { %asmtmp = tail call i64 asm "bswapq ${0:q}", "=r,0,~{dirflag},~{fpsr},~{flags}"(i64 %x) nounwind ret i64 %asmtmp } +define i32 @pen(i32 %x) nounwind { + %asmtmp = tail call i32 asm "bswapl ${0:q}", "=r,0,~{dirflag},~{fpsr},~{flags}"(i32 %x) nounwind + ret i32 %asmtmp +}