Re: [llvm-commits] PR1350 patch 2

2007-06-16 Thread Christopher Lamb
Whoops! Omitted a hunk.On Jun 16, 2007, at 9:14 PM, Christopher Lamb wrote:This actually adds some new SDNodes, and I want to make sure I've done it right. After this patch the SDNodes are there, but ScheduleDAG will assert out if they're used. No dg regressions that I can tell from this patch (the

Re: [llvm-commits] PR1350 patch 2

2007-06-16 Thread Christopher Lamb
Whoops! Omitted a hunk. On Jun 16, 2007, at 9:14 PM, Christopher Lamb wrote: This actually adds some new SDNodes, and I want to make sure I've done it right. After this patch the SDNodes are there, but ScheduleDAG will assert out if they're used. No dg regressions that I can tell from this

[llvm-commits] PR1350 patch 2

2007-06-16 Thread Christopher Lamb
This actually adds some new SDNodes, and I want to make sure I've done it right. After this patch the SDNodes are there, but ScheduleDAG will assert out if they're used. No dg regressions that I can tell from this patch (the nodes shouldn't get created unless a backed does so explicitly). Give me t

[llvm-commits] CVS: llvm/test/CodeGen/X86/2007-06-15-IntToMMX.ll

2007-06-16 Thread Bill Wendling
Changes in directory llvm/test/CodeGen/X86: 2007-06-15-IntToMMX.ll updated: 1.1 -> 1.2 --- Log message: XFAILing until I can fix properly. --- Diffs of the changes: (+1 -0) 2007-06-15-IntToMMX.ll |1 + 1 files changed, 1 insertion(+) Index: llvm/test/CodeGen/X86/2007-06-15-IntToMMX.l

Re: [llvm-commits] CVS: llvm/lib/Target/X86/X86ISelLowering.cpp

2007-06-16 Thread Bill Wendling
Doh! Reverting. Sorry about that. -bw On Jun 16, 2007, at 12:35 PM, Chris Lattner wrote: > On Jun 15, 2007, at 11:17 PM, Bill Wendling wrote: >> Fix a failure to bit_convert from integer GPR to MMX register. > > Bill, this causes a regression on this testcase: > > define double @test2(i64 %A) {

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

2007-06-16 Thread Bill Wendling
Changes in directory llvm/lib/Target/X86: X86ISelLowering.cpp updated: 1.410 -> 1.411 --- Log message: Revert patch. It regresses: define double @test2(i64 %A) { %B = bitcast i64 %A to double ret double %B } $ llvm-as < t.ll | llc -march=x86-64 before: .align 4 .glo

[llvm-commits] CVS: llvm/tools/llvm2cpp/CppWriter.cpp

2007-06-16 Thread Reid Spencer
Changes in directory llvm/tools/llvm2cpp: CppWriter.cpp updated: 1.51 -> 1.52 --- Log message: Fix PR1517: http://llvm.org/PR1517 : Use SmallPtrSet instead of std::vector to eliminate duplicate uses in a function generated with -gen-function. This prevents the output from having multiple duplic

[llvm-commits] CVS: llvm/tools/llvm2cpp/CppWriter.cpp

2007-06-16 Thread Reid Spencer
Changes in directory llvm/tools/llvm2cpp: CppWriter.cpp updated: 1.50 -> 1.51 --- Log message: Fix PR1516: http://llvm.org/PR1516 : When printing the uses of a function (-gen-function only), make sure to include the constants referenced by intializers of global variables. --- Diffs of the ch

Re: [llvm-commits] CVS: llvm/lib/Target/X86/X86ISelLowering.cpp

2007-06-16 Thread Chris Lattner
On Jun 15, 2007, at 11:17 PM, Bill Wendling wrote: > Fix a failure to bit_convert from integer GPR to MMX register. Bill, this causes a regression on this testcase: define double @test2(i64 %A) { %B = bitcast i64 %A to double ret double %B } $ llvm-as < t.ll | llc -march=x86-64 before:

[llvm-commits] CVS: llvm/tools/llvm2cpp/CppWriter.cpp

2007-06-16 Thread Nick Lewycky
Changes in directory llvm/tools/llvm2cpp: CppWriter.cpp updated: 1.49 -> 1.50 --- Log message: Call the correct function name. Patch from Stephane Letz. --- Diffs of the changes: (+1 -1) CppWriter.cpp |2 +- 1 files changed, 1 insertion(+), 1 deletion(-) Index: llvm/tools/llvm2cpp/Cp

[llvm-commits] CVS: llvm/test/Other/2007-06-16-Funcname.ll

2007-06-16 Thread Nick Lewycky
Changes in directory llvm/test/Other: 2007-06-16-Funcname.ll added (r1.1) --- Log message: Call the correct function name. Patch from Stephane Letz. --- Diffs of the changes: (+7 -0) 2007-06-16-Funcname.ll |7 +++ 1 files changed, 7 insertions(+) Index: llvm/test/Other/2007-06-16

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

2007-06-16 Thread Evan Cheng
Changes in directory llvm/lib/CodeGen: IfConversion.cpp updated: 1.54 -> 1.55 --- Log message: Really turn if-converter loose: 1. Consider all possible ifcvt cases at once. No longer restricted to bottom up iterative approach. 2. Sort all possible cases based on a cost function. Perform the

Re: [llvm-commits] [128471] Add annotate attribute.

2007-06-16 Thread Duncan Sands
Hi, > + // Add llvm.global.annotations > + if (!AttributeAnnotateGlobals.empty()) { > +std::vector AttrList; since you know how big this vector is going to be, you could reserve that much space here. > +for (unsigned i = 0, e = AttributeAnnotateGlobals.size(); i != e; ++i) { > + C