[llvm-commits] [llvm] r54148 - in /llvm/trunk: include/llvm/CodeGen/SelectionDAGNodes.h lib/Target/X86/X86Instr64bit.td lib/Target/X86/X86InstrInfo.td test/CodeGen/X86/zext-inreg-0.ll test/CodeGen/X86/zext-inreg-1.ll
Dan Gohman
gohman at apple.com
Mon Jul 28 18:02:18 PDT 2008
Author: djg
Date: Mon Jul 28 20:02:18 2008
New Revision: 54148
URL: http://llvm.org/viewvc/llvm-project?rev=54148&view=rev
Log:
Revert 54147.
Removed:
llvm/trunk/test/CodeGen/X86/zext-inreg-0.ll
llvm/trunk/test/CodeGen/X86/zext-inreg-1.ll
Modified:
llvm/trunk/include/llvm/CodeGen/SelectionDAGNodes.h
llvm/trunk/lib/Target/X86/X86Instr64bit.td
llvm/trunk/lib/Target/X86/X86InstrInfo.td
Modified: llvm/trunk/include/llvm/CodeGen/SelectionDAGNodes.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/CodeGen/SelectionDAGNodes.h?rev=54148&r1=54147&r2=54148&view=diff
==============================================================================
--- llvm/trunk/include/llvm/CodeGen/SelectionDAGNodes.h (original)
+++ llvm/trunk/include/llvm/CodeGen/SelectionDAGNodes.h Mon Jul 28 20:02:18 2008
@@ -377,8 +377,6 @@
// extending the low 8 bits of a 32-bit register to fill the top 24 bits
// with the 7th bit). The size of the smaller type is indicated by the 1th
// operand, a ValueType node.
- // Note that there is intentionally no corresponding ZERO_EXTEND_INREG; an
- // AND with an appropriate constant is used instead.
SIGN_EXTEND_INREG,
/// FP_TO_[US]INT - Convert a floating point value to a signed or unsigned
Modified: llvm/trunk/lib/Target/X86/X86Instr64bit.td
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86Instr64bit.td?rev=54148&r1=54147&r2=54148&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86Instr64bit.td (original)
+++ llvm/trunk/lib/Target/X86/X86Instr64bit.td Mon Jul 28 20:02:18 2008
@@ -1240,12 +1240,6 @@
(SUBREG_TO_REG (i64 0),
(i32 (EXTRACT_SUBREG GR64:$src, x86_subreg_32bit)),
x86_subreg_32bit)>;
-// r & (2^16-1) ==> movz
-def : Pat<(and GR64:$src, 0xffff),
- (MOVZX64rr16 (i16 (EXTRACT_SUBREG GR64:$src, x86_subreg_16bit)))>;
-// r & (2^8-1) ==> movz
-def : Pat<(and GR64:$src, 0xff),
- (MOVZX64rr8 (i8 (EXTRACT_SUBREG GR64:$src, x86_subreg_8bit)))>;
// (shl x, 1) ==> (add x, x)
def : Pat<(shl GR64:$src1, (i8 1)), (ADD64rr GR64:$src1, GR64:$src1)>;
Modified: llvm/trunk/lib/Target/X86/X86InstrInfo.td
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86InstrInfo.td?rev=54148&r1=54147&r2=54148&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86InstrInfo.td (original)
+++ llvm/trunk/lib/Target/X86/X86InstrInfo.td Mon Jul 28 20:02:18 2008
@@ -2763,16 +2763,6 @@
// Some peepholes
//===----------------------------------------------------------------------===//
-// r & (2^16-1) ==> movz
-def : Pat<(and GR32:$src1, 0xffff),
- (MOVZX32rr16 (i16 (EXTRACT_SUBREG GR32:$src1, x86_subreg_16bit)))>;
-// r & (2^8-1) ==> movz
-def : Pat<(and GR32:$src1, 0xff),
- (MOVZX32rr8 (i8 (EXTRACT_SUBREG GR32:$src1, x86_subreg_8bit)))>;
-// r & (2^8-1) ==> movz
-def : Pat<(and GR16:$src1, 0xff),
- (MOVZX16rr8 (i8 (EXTRACT_SUBREG GR16:$src1, x86_subreg_8bit)))>;
-
// (shl x, 1) ==> (add x, x)
def : Pat<(shl GR8 :$src1, (i8 1)), (ADD8rr GR8 :$src1, GR8 :$src1)>;
def : Pat<(shl GR16:$src1, (i8 1)), (ADD16rr GR16:$src1, GR16:$src1)>;
Removed: llvm/trunk/test/CodeGen/X86/zext-inreg-0.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/X86/zext-inreg-0.ll?rev=54147&view=auto
==============================================================================
--- llvm/trunk/test/CodeGen/X86/zext-inreg-0.ll (original)
+++ llvm/trunk/test/CodeGen/X86/zext-inreg-0.ll (removed)
@@ -1,62 +0,0 @@
-; RUN: llvm-as < %s | llc -march=x86 | not grep and
-; RUN: llvm-as < %s | llc -march=x86-64 | not grep and
-
-; These should use movzbl instead of 'and 255'.
-; This related to not having a ZERO_EXTEND_REG opcode.
-
-define i32 @a(i32 %d) nounwind {
- %e = add i32 %d, 1
- %retval = and i32 %e, 255
- ret i32 %retval
-}
-define i32 @b(float %d) nounwind {
- %tmp12 = fptoui float %d to i8
- %retval = zext i8 %tmp12 to i32
- ret i32 %retval
-}
-define i32 @c(i32 %d) nounwind {
- %e = add i32 %d, 1
- %retval = and i32 %e, 65535
- ret i32 %retval
-}
-define i64 @d(i64 %d) nounwind {
- %e = add i64 %d, 1
- %retval = and i64 %e, 255
- ret i64 %retval
-}
-define i64 @e(i64 %d) nounwind {
- %e = add i64 %d, 1
- %retval = and i64 %e, 65535
- ret i64 %retval
-}
-define i64 @f(i64 %d) nounwind {
- %e = add i64 %d, 1
- %retval = and i64 %e, 4294967295
- ret i64 %retval
-}
-
-define i32 @g(i8 %d) nounwind {
- %e = add i8 %d, 1
- %retval = zext i8 %e to i32
- ret i32 %retval
-}
-define i32 @h(i16 %d) nounwind {
- %e = add i16 %d, 1
- %retval = zext i16 %e to i32
- ret i32 %retval
-}
-define i64 @i(i8 %d) nounwind {
- %e = add i8 %d, 1
- %retval = zext i8 %e to i64
- ret i64 %retval
-}
-define i64 @j(i16 %d) nounwind {
- %e = add i16 %d, 1
- %retval = zext i16 %e to i64
- ret i64 %retval
-}
-define i64 @k(i32 %d) nounwind {
- %e = add i32 %d, 1
- %retval = zext i32 %e to i64
- ret i64 %retval
-}
Removed: llvm/trunk/test/CodeGen/X86/zext-inreg-1.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/X86/zext-inreg-1.ll?rev=54147&view=auto
==============================================================================
--- llvm/trunk/test/CodeGen/X86/zext-inreg-1.ll (original)
+++ llvm/trunk/test/CodeGen/X86/zext-inreg-1.ll (removed)
@@ -1,18 +0,0 @@
-; RUN: llvm-as < %s | llc -march=x86 | not grep and
-
-; These tests differ from the ones in zext-inreg-0.ll in that
-; on x86-64 they do require and instructions.
-
-; These should use movzbl instead of 'and 255'.
-; This related to not having ZERO_EXTEND_REG node.
-
-define i64 @g(i64 %d) nounwind {
- %e = add i64 %d, 1
- %retval = and i64 %e, 1099511627775
- ret i64 %retval
-}
-define i64 @h(i64 %d) nounwind {
- %e = add i64 %d, 1
- %retval = and i64 %e, 281474976710655
- ret i64 %retval
-}
More information about the llvm-commits
mailing list