Re: [llvm-commits] [125160] Better support for variable size struct fields.

2007-03-25 Thread Duncan Sands
> Is this already fixed? I am testing a fix. Ciao, Duncan. ___ llvm-commits mailing list llvm-commits@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

[llvm-commits] CVS: llvm/lib/Target/MSIL/Makefile

2007-03-25 Thread Anton Korobeynikov
Changes in directory llvm/lib/Target/MSIL: Makefile updated: 1.1 -> 1.2 --- Log message: Fix authorship --- Diffs of the changes: (+1 -1) Makefile |2 +- 1 files changed, 1 insertion(+), 1 deletion(-) Index: llvm/lib/Target/MSIL/Makefile diff -u llvm/lib/Target/MSIL/Makefile:1.1 llvm

[llvm-commits] CVS: llvm/include/llvm/CodeGen/SelectionDAGISel.h

2007-03-25 Thread Anton Korobeynikov
Changes in directory llvm/include/llvm/CodeGen: SelectionDAGISel.h updated: 1.34 -> 1.35 --- Log message: First step of switch lowering refactoring: perform worklist-driven strategy, emit JT's where possible. --- Diffs of the changes: (+24 -12) SelectionDAGISel.h | 36 ++

[llvm-commits] CVS: llvm/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp

2007-03-25 Thread Anton Korobeynikov
Changes in directory llvm/lib/CodeGen/SelectionDAG: SelectionDAGISel.cpp updated: 1.393 -> 1.394 --- Log message: First step of switch lowering refactoring: perform worklist-driven strategy, emit JT's where possible. --- Diffs of the changes: (+253 -180) SelectionDAGISel.cpp | 433 ++

Re: [llvm-commits] CVS: llvm/lib/Transforms/Scalar/InstructionCombining.cpp

2007-03-25 Thread Chris Lattner
> Index: llvm/lib/Transforms/Scalar/InstructionCombining.cpp > diff -u llvm/lib/Transforms/Scalar/InstructionCombining.cpp:1.687 > llvm/lib/Transforms/Scalar/InstructionCombining.cpp:1.688 > --- llvm/lib/Transforms/Scalar/InstructionCombining.cpp:1.687 Sun > Mar 25 00:01:29 2007 > +++ llvm/lib/

Re: [llvm-commits] CVS: llvm/lib/Transforms/Scalar/InstructionCombining.cpp

2007-03-25 Thread Reid Spencer
On Sun, 2007-03-25 at 11:55 -0700, Chris Lattner wrote: > > Index: llvm/lib/Transforms/Scalar/InstructionCombining.cpp > > diff -u llvm/lib/Transforms/Scalar/InstructionCombining.cpp:1.687 > > llvm/lib/Transforms/Scalar/InstructionCombining.cpp:1.688 > > --- llvm/lib/Transforms/Scalar/Instruction

Re: [llvm-commits] CVS: llvm/lib/Transforms/Scalar/InstructionCombining.cpp

2007-03-25 Thread Chris Lattner
On Mar 25, 2007, at 12:10 PM, Reid Spencer wrote: > On Sun, 2007-03-25 at 11:55 -0700, Chris Lattner wrote: >>> Index: llvm/lib/Transforms/Scalar/InstructionCombining.cpp >>> diff -u llvm/lib/Transforms/Scalar/InstructionCombining.cpp:1.687 >>> llvm/lib/Transforms/Scalar/InstructionCombining.cpp:

Re: [llvm-commits] CVS: llvm/lib/Transforms/Scalar/InstructionCombining.cpp

2007-03-25 Thread Reid Spencer
On Sun, 2007-03-25 at 12:25 -0700, Chris Lattner wrote: > > > >>> +/// SubOne - Subtract one from a ConstantInt > >>> static ConstantInt *SubOne(ConstantInt *C) { > >> > >> Shouldn't these use ++/-- on APInt? That seems more efficient. > > > > I should really have these functions declare the para

Re: [llvm-commits] CVS: llvm/lib/Transforms/Scalar/InstructionCombining.cpp

2007-03-25 Thread Chris Lattner
>> Right. ConstantInt's are immutable, so it doesn't really need to be >> marked const. I'm saying that the implementation of these methods >> shouldn't build a "1" apint, then add it. Instead, it should just >> increment an apint with ++. > > Yup. I've already changed it to: > > static Constant

[llvm-commits] CVS: llvm/lib/Transforms/Scalar/InstructionCombining.cpp

2007-03-25 Thread Reid Spencer
Changes in directory llvm/lib/Transforms/Scalar: InstructionCombining.cpp updated: 1.688 -> 1.689 --- Log message: Some cleanup from review: * Don't assume shift amounts are <= 64 bits * Avoid creating an extra APInt in SubOne and AddOne by using -- and ++ * Add another use of getLowBitsSet * C

Re: [llvm-commits] CVS: llvm/lib/Transforms/Scalar/InstructionCombining.cpp

2007-03-25 Thread Chris Lattner
> Index: llvm/lib/Transforms/Scalar/InstructionCombining.cpp > diff -u llvm/lib/Transforms/Scalar/InstructionCombining.cpp:1.688 > llvm/lib/Transforms/Scalar/InstructionCombining.cpp:1.689 > --- llvm/lib/Transforms/Scalar/InstructionCombining.cpp:1.688 Sun > Mar 25 00:33:51 2007 > +++ llvm/lib/

[llvm-commits] CVS: llvm/test/Transforms/InstCombine/cast2.ll getelementptr.ll

2007-03-25 Thread Chris Lattner
Changes in directory llvm/test/Transforms/InstCombine: cast2.ll updated: 1.1 -> 1.2 getelementptr.ll updated: 1.19 -> 1.20 --- Log message: new testcase --- Diffs of the changes: (+8 -1) cast2.ll |6 ++ getelementptr.ll |3 ++- 2 files changed, 8 insertions(+), 1 deleti

[llvm-commits] CVS: llvm/lib/Transforms/Scalar/InstructionCombining.cpp

2007-03-25 Thread Chris Lattner
Changes in directory llvm/lib/Transforms/Scalar: InstructionCombining.cpp updated: 1.689 -> 1.690 --- Log message: implement Transforms/InstCombine/cast2.ll:test3 and PR1263: http://llvm.org/PR1263 --- Diffs of the changes: (+21 -1) InstructionCombining.cpp | 22 +-

[llvm-commits] CVS: llvm/lib/Transforms/Scalar/InstructionCombining.cpp

2007-03-25 Thread Reid Spencer
Changes in directory llvm/lib/Transforms/Scalar: InstructionCombining.cpp updated: 1.690 -> 1.691 --- Log message: For PR1271: http://llvm.org/PR1271 : Remove a use of getLowBitsSet that caused the mask used for replacement of shl/lshr pairs with an AND instruction to be computed incorrectly. I

[llvm-commits] CVS: llvm/test/Transforms/InstCombine/2007-03-25-DoubleShift.ll

2007-03-25 Thread Reid Spencer
Changes in directory llvm/test/Transforms/InstCombine: 2007-03-25-DoubleShift.ll added (r1.1) --- Log message: Add a test case for PR1271: http://llvm.org/PR1271 (necessary, but not sufficient). --- Diffs of the changes: (+9 -0) 2007-03-25-DoubleShift.ll |9 + 1 files changed

[llvm-commits] CVS: llvm/include/llvm/ADT/APInt.h

2007-03-25 Thread Reid Spencer
Changes in directory llvm/include/llvm/ADT: APInt.h updated: 1.57 -> 1.58 --- Log message: Compute getLowBitsSet correctly. Using the complement of a 64-bit value and shifting down without regard for the bitwidth of the APInt can lead to incorrect initialization values. Instead, check for the w

[llvm-commits] CVS: llvm/test/CodeGen/Generic/switch-lower-feature.ll

2007-03-25 Thread Anton Korobeynikov
Changes in directory llvm/test/CodeGen/Generic: switch-lower-feature.ll added (r1.1) --- Log message: Add "feature" testcase for new switch lowering code --- Diffs of the changes: (+32 -0) switch-lower-feature.ll | 32 1 files changed, 32 insertions(+)

[llvm-commits] CVS: llvm/include/llvm/ValueSymbolTable.h

2007-03-25 Thread Chris Lattner
Changes in directory llvm/include/llvm: ValueSymbolTable.h updated: 1.8 -> 1.9 --- Log message: Start value symbol tables out small (space for 16 elts), not huge (space for 512). This is particularly useful for the JIT, which lazily deserializes functions. --- Diffs of the changes: (+1 -1)

[llvm-commits] CVS: llvm/lib/Transforms/Scalar/LoopStrengthReduce.cpp

2007-03-25 Thread Dale Johannesen
Changes in directory llvm/lib/Transforms/Scalar: LoopStrengthReduce.cpp updated: 1.121 -> 1.122 --- Log message: Look through bitcast when finding IVs. (Chris' patch really.) --- Diffs of the changes: (+10 -0) LoopStrengthReduce.cpp | 10 ++ 1 files changed, 10 insertions(+) I

[llvm-commits] CVS: llvm/lib/Transforms/Scalar/InstructionCombining.cpp

2007-03-25 Thread Reid Spencer
Changes in directory llvm/lib/Transforms/Scalar: InstructionCombining.cpp updated: 1.691 -> 1.692 --- Log message: For PR1271: http://llvm.org/PR1271 : Fix SingleSource/Regression/C/2003-05-21-UnionBitFields.c by changing a getHighBitsSet call to getLowBitsSet call that was incorrectly converte

[llvm-commits] CVS: llvm/test/Transforms/InstCombine/2007-03-25-BadShiftMask.ll

2007-03-25 Thread Reid Spencer
Changes in directory llvm/test/Transforms/InstCombine: 2007-03-25-BadShiftMask.ll added (r1.1) --- Log message: Test case for PR1271: http://llvm.org/PR1271 involving construction of a bad mask to replace a shift instruction. --- Diffs of the changes: (+28 -0) 2007-03-25-BadShiftMask.ll

[llvm-commits] CVS: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp

2007-03-25 Thread Evan Cheng
Changes in directory llvm/lib/CodeGen/SelectionDAG: DAGCombiner.cpp updated: 1.281 -> 1.282 --- Log message: SIGN_EXTEND_INREG requires one extra operand, a ValueType node. --- Diffs of the changes: (+6 -2) DAGCombiner.cpp |8 ++-- 1 files changed, 6 insertions(+), 2 deletions(-)

[llvm-commits] CVS: llvm/utils/TableGen/CodeGenTarget.cpp

2007-03-25 Thread Bill Wendling
Changes in directory llvm/utils/TableGen: CodeGenTarget.cpp updated: 1.86 -> 1.87 --- Log message: Add support for the v1i64 type. This makes better code for this: #include extern __m64 C; void baz(__v2si *A, __v2si *B) { *A = C; _mm_empty(); } We get this: _baz: call "L1$pb"

[llvm-commits] CVS: llvm/include/llvm/Intrinsics.td

2007-03-25 Thread Bill Wendling
Changes in directory llvm/include/llvm: Intrinsics.td updated: 1.50 -> 1.51 --- Log message: Add support for the v1i64 type. This makes better code for this: #include extern __m64 C; void baz(__v2si *A, __v2si *B) { *A = C; _mm_empty(); } We get this: _baz: call "L1$pb" "L1$pb

[llvm-commits] CVS: llvm/lib/Target/X86/X86ISelLowering.cpp X86InstrMMX.td X86RegisterInfo.td

2007-03-25 Thread Bill Wendling
Changes in directory llvm/lib/Target/X86: X86ISelLowering.cpp updated: 1.377 -> 1.378 X86InstrMMX.td updated: 1.24 -> 1.25 X86RegisterInfo.td updated: 1.40 -> 1.41 --- Log message: Add support for the v1i64 type. This makes better code for this: #include extern __m64 C; void baz(__v2si *A,

[llvm-commits] CVS: llvm/lib/Target/X86/README-MMX.txt

2007-03-25 Thread Bill Wendling
Changes in directory llvm/lib/Target/X86: README-MMX.txt updated: 1.1 -> 1.2 --- Log message: Updated. --- Diffs of the changes: (+0 -54) README-MMX.txt | 54 -- 1 files changed, 54 deletions(-) Index: llvm/lib/Target/X86/README-MMX.t